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.

Sign up for GitHub

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

Jump to bottom

Wrong Ordering in AwsCredentialsProvider.new_profile() #301

Closed
aihaddad opened this issue Jun 24, 2021 · 1 comment
Closed

Wrong Ordering in AwsCredentialsProvider.new_profile() #301

aihaddad opened this issue Jun 24, 2021 · 1 comment
Labels
bug This issue is a bug. CRT/SDK p2 This is a standard priority issue

Comments

@aihaddad
Copy link

According to the docstring, the profile name order should be as follows:

  1. profile_name I provide
  2. AWS_PROFILE env var
  3. default
  • I have an AWS_PROFILE set, but I pass another profile_name, and it still uses AWS_PROFILE
  • I unset AWS_PROFILE and it works

profile_name needs to over-ride the environment variable.
@jmklix jmklix added the CRT/SDK label Mar 27, 2023
@jmklix jmklix added bug This issue is a bug. p2 This is a standard priority issue labels Nov 15, 2023
@jmklix
Copy link
Member

jmklix commented Mar 28, 2024

This is fixed with this PR: awslabs/aws-c-sdkutils#30

@jmklix jmklix closed this as completed Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug This issue is a bug. CRT/SDK p2 This is a standard priority issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@aihaddad @jmklix