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

Add support for MFA, related to issue #58 #59

Merged

ncatelli merged 1 commit into disneystreaming:develop from ledor473:develop

Oct 27, 2021

Contributor

ledor473 commented Oct 25, 2021

Fixes #58

When running the previous release:

$ ssm session --region us-east-1 --profile <REDACTED> --instance <REDACTED>
FATAL   Error when trying to create session:
AssumeRoleTokenProviderNotSetError: assume role with MFA enabled, but AssumeRoleTokenProvider session option not set.

When running with this PR:

$ ./ssm session --region us-east-1 --profile <REDACTED> --instance <REDACTED>
Assume Role MFA token code: <REDACTED>
INFO    Retrieved 10 usable instances.
       Instance ID          Region     Profile
? Showing 10/820 instances. Make a Selection:  [Use arrows to move, space to select, type to filter]


          Add support for MFA, related to issue disneystreaming#58

586776e

ncatelli approved these changes

View reviewed changes

ncatelli left a comment

Looks good to me, thanks for the contribution.

ncatelli merged commit a0c0299 into disneystreaming:develop

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet