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

Improve EC2 instance role and EC2Metadata client error messages #548

Merged
merged 2 commits into from
Feb 8, 2016

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented Feb 8, 2016

Improves the error messages received for EC2Metadata client requests.
The error message bodies were being dropped by the SDK and are now
included in the error returned.

Also updates the SDK to returned errors to be chained in a stack so that
nested errors are not ignored. This change updates the SDK so that the
chain of errors which occurred are all included. Instead of just the first error.

Fix #545

Improves the error messages received for EC2Metadata client requests.
The error message bodies were being dropped by the SDK and are now
included in the error returned.

Fix #545
Allow returned errors to be chained in a stack so that nested errors are
not ignored. This change updates the SDK so that the chain of errors
which occurred are all included. Instead of just the first error.
@xibz
Copy link
Contributor

xibz commented Feb 8, 2016

Looks good! :shipit:

jasdel added a commit that referenced this pull request Feb 8, 2016
Improve EC2 instance role and EC2Metadata client error messages
@jasdel jasdel merged commit 38ff93f into master Feb 8, 2016
xibz added a commit that referenced this pull request Feb 9, 2016
@jasdel jasdel deleted the i545FixmetadataError branch February 16, 2016 18:13
skotambkar pushed a commit to skotambkar/aws-sdk-go that referenced this pull request May 20, 2021
Services
---
* Synced the V2 SDK with latest AWS service API definitions.

Breaking Changes
---
* Removed prototype and experiment types from master branch of SDK.
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