Merge pull request #27493 from cuppett/cuppett/simplify-aws-credential-provider

This commit is contained in:
Julius Härtl 2021-12-21 11:58:52 +01:00 committed by GitHub
commit 1acfbd0b06
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -109,15 +109,11 @@ trait S3ConnectionTrait {
$base_url = $scheme . '://' . $this->params['hostname'] . ':' . $this->params['port'] . '/';
// Adding explicit credential provider to the beginning chain.
// Including environment variables and IAM instance profiles.
// Including default credential provider (skipping AWS shared config files).
$provider = CredentialProvider::memoize(
CredentialProvider::chain(
$this->paramCredentialProvider(),
CredentialProvider::env(),
CredentialProvider::assumeRoleWithWebIdentityCredentialProvider(),
!empty(getenv(EcsCredentialProvider::ENV_URI))
? CredentialProvider::ecsCredentials()
: CredentialProvider::instanceProfile()
CredentialProvider::defaultProvider(['use_aws_shared_config_files' => false])
)
);