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

if present, copy 'region' from long-term to short-term config #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jantman
Copy link

@jantman jantman commented Dec 20, 2017

This is a wonderful tool, thanks so much! It does really simplify using MFA-enabled API access.

This is mainly a convenience; if the region option is set in the long-term config section, it copies it to the short term section before writing out the config file.

Copy link
Owner

@broamski broamski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As you can see from the failing test, long_term_name is not accessible from this location.

@jantman
Copy link
Author

jantman commented Dec 23, 2017

@broamski Apologies for that.

Bug fixed, new commit pushed that passes flake8 and works correctly locally.

andrewwheal pushed a commit to andrewwheal/aws-mfa that referenced this pull request Oct 22, 2019
@chrisandrews7
Copy link

@broamski Would be great to have this please.

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

Successfully merging this pull request may close these issues.

3 participants