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

Throw an error if an incomplete profile is encountered #690

Closed
jamesls opened this issue Mar 5, 2014 · 0 comments · Fixed by boto/botocore#331
Closed

Throw an error if an incomplete profile is encountered #690

jamesls opened this issue Mar 5, 2014 · 0 comments · Fixed by boto/botocore#331
Labels
bug This issue is a bug.

Comments

@jamesls
Copy link
Member

jamesls commented Mar 5, 2014

This is along the same lines as #133
If a user has a profile configured but it is missing keys, we fall back to other credential mechanisms. For example, if I have a config file of:

[profile foo]
aws_access_key_id=foo
aws_secret_access_MISPELLED=bar

And I run aws service op --profile foo, then we'll silently fall back to other credentials provider. If I had a valid ~/.boto config, then we'd silently use the boto config, which might have surprising results.

I think we should update the CLI in the same spirit as #133. If you specify a --profile, it has to exist in its entirety, otherwise an error is thrown. This means we should at least require access/secret key entries in the profile.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant