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

Handle case when create-virtual-mfa-device errors out #1447

Merged
merged 1 commit into from
Aug 14, 2015

Conversation

jamesls
Copy link
Member

@jamesls jamesls commented Aug 13, 2015

This was previously propogating a KeyError and would just
display the error message 'VirtualMFADevice'. Now
we let the error propogate and print a full error message.

cc @kyleknap @mtdowling @rayluo

This was previously propogating a KeyError and would just
display the error message 'VirtualMFADevice'.  Now
we let the error propogate and print a full error message.
@kyleknap
Copy link
Contributor

Looks good. 🚢

@jamesls jamesls merged commit 0b5583a into aws:develop Aug 14, 2015
jamesls added a commit that referenced this pull request Aug 14, 2015
Closes #1447

* fix-mfa-device:
  Handle case when create-virtual-mfa-device errors out
jamesls added a commit that referenced this pull request Aug 14, 2015
rayluo pushed a commit that referenced this pull request Aug 20, 2015
* release-1.7.46:
  Bumping version to 1.7.46
  Update changelog
  Fix pep8 issue
  Support nested hashes/lists
  Add more comments to make backtracking more clear
  Add consume_whitespace arg to expect
  Missing values parse to empty string
  Handle newlines with appropriate errors
  fixed typo
  Update changelog with fix from #1447
  Handle case when create-virtual-mfa-device errors out
  Remove 'ur' and use 'u' strings instead
  Add support for hash literals
  Initial commit of imporoved shorthand parser
thoward-godaddy pushed a commit to thoward-godaddy/aws-cli that referenced this pull request Feb 12, 2022
Adds SageMaker Ground Truth Lambda events
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.

2 participants