fallintoplace opened a new pull request, #1574:
URL: https://github.com/apache/iceberg-go/pull/1574

   ## What changed
   
   Reject incomplete static AWS credential properties in Glue and S3 
configuration. Access key and secret key must now be provided together, and a 
session token cannot be configured without that pair.
   
   ## Why
   
   Providing any single credential field previously replaced the default AWS 
provider chain with invalid static credentials containing blank fields. This 
delayed a configuration error until an AWS request and could obscure the actual 
typo.
   
   Complete key pairs, with or without a session token, continue to use the 
static provider.
   
   ## Testing
   
   - `go test ./catalog/glue -run 
TestLoadAWSConfigRejectsIncompleteStaticCredentials`
   - `go test ./io/gocloud -run 
'TestParseAWSConfigRejectsIncompleteStaticCredentials|TestResolveS3AWSConfigCredentialPrecedence'`
   - `go vet ./catalog/glue ./io/gocloud`


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to