expose a constructor func for awserr.Error types #254
Labels
feature-request
A feature should be added or improved.
Comments
|
Thanks for posting this @sclasen. An additional place this pain point is returning awserr.Error compatible error when using custom credentials providers. If we limit to the Code/Message/OrigErr fields then the New(...) constructor could be exposed via aws/awserr package. Though the awserr.RequestFailure interface would still present a pain point for the same reasons as awserr.Error. A awserr.NewRequestFailure(...) Would also make sense. |
|
👍 |
jasdel
added a commit
that referenced
this issue
Jun 12, 2015
Adds two new constructors for the awserr Error types. Can also be used for building awserr.Error for testing. - awserr.New() returns an Error object described by code, message, and origErr. If origErr satisfies the Error interface it will not be wrapped within a new Error object, and will instead be returned. - awserr.NewRequestFailure() returns a new request error wrapper for the given Error provided. Fixes: #254
lsegal
added a commit
that referenced
this issue
Jun 18, 2015
|
Thanks for suggesting this feature @sclasen. Now that the change has been pushed I'm going to go ahead and close this issue. Please let open an issue, if you have any further feedback, or issues. |
skotambkar
pushed a commit
to skotambkar/aws-sdk-go
that referenced
this issue
May 20, 2021
Updates the v2 SDK to use the latest go-jmespath. Fix aws#247
skotambkar
pushed a commit
to skotambkar/aws-sdk-go
that referenced
this issue
May 20, 2021
Services --- * Synced the V2 SDK with latests AWS service API definitions. SDK Enhancements --- * deps: Update SDK to latest go-jmespath ([aws#254](aws/aws-sdk-go-v2#254)) SDK Bugs --- * `internal/ini`: Fix bug on trimming rhs spaces closes ([aws#260](aws/aws-sdk-go-v2#260)) * Fixes a bug trimming RHS spaces not being read correctly from the ini file. * Fix [aws#259](aws/aws-sdk-go-v2#259)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
internal.apierr.New(...)should probably have a non-internal counterpart. otherwise it is quite painful to create errors to test error handling withThe text was updated successfully, but these errors were encountered: