Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AWS Default Credentials Chain not fetched in > 2.0.0 #233

Open
nishkrishnan opened this issue Mar 7, 2022 · 2 comments · May be fixed by #281
Open

AWS Default Credentials Chain not fetched in > 2.0.0 #233

nishkrishnan opened this issue Mar 7, 2022 · 2 comments · May be fixed by #281
Labels
enhancement pinned Disable auto stale

Comments

@nishkrishnan
Copy link

nishkrishnan commented Mar 7, 2022

As of this PR:
#177

AWS Default Credentials Provider is never used so now we're forced to either pass in an access key/secret key or pass in a role to assume. However, the default credentials provider is much more flexible:
https://aws.github.io/aws-sdk-go-v2/docs/configuring-sdk/#specifying-credentials

In our case, we use web identity auth which is now broken.

@nishkrishnan nishkrishnan changed the title AWS Credentials not working in > 2.0.0 AWS Default Credentials not fetched in > 2.0.0 Mar 7, 2022
@nishkrishnan nishkrishnan changed the title AWS Default Credentials not fetched in > 2.0.0 AWS Default Credentials Chain not fetched in > 2.0.0 Mar 7, 2022
@hbollon
Copy link
Member

hbollon commented Mar 8, 2022

Hey @nishkrishnan !
Thanks for your issue, I plan to migrate from aws-sdk-go to aws-sdk-go-v2 in the near future. I will replace NewSession() by LoadDefaultConfig(), it should resolve your case.

@hbollon hbollon added pinned Disable auto stale enhancement labels Apr 25, 2022
@chrispruitt chrispruitt linked a pull request Nov 8, 2023 that will close this issue
@chrispruitt
Copy link

@hbollon any concerns with the linked pull request? This will fix the issue until you update the AWS SDK and is backward compatible. Also, happy to adjust if you see an issue with it. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement pinned Disable auto stale
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants