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

Add raw error message bytes to SerializationError errors #2600

Merged
merged 1 commit into from
May 17, 2019

Conversation

jasdel
Copy link
Contributor

@jasdel jasdel commented May 14, 2019

Updates the SDK's API error message SerializationError handling to
capture the original error message byte, and include it in the
SerializationError error value.

Also replaces the "SerializationError" literal string in the SDK with the constant, ErrCodeSerialization.

Fix #2562, #2411, #2315

Updates the SDK's API error message SerializationError handling to
capture the original error message byte, and include it in the
SerializationError error value.

Related to aws#2562, aws#2411, aws#2315
@jasdel jasdel merged commit 3c8d61f into aws:master May 17, 2019
@jasdel jasdel deleted the feat/ErrorDetails branch May 17, 2019 21:54
jasdel added a commit that referenced this pull request May 17, 2019
Updates the pending change log with PR #2600.
@aws-sdk-go-automation aws-sdk-go-automation mentioned this pull request May 20, 2019
diehlaws pushed a commit to diehlaws/aws-sdk-go that referenced this pull request Jul 2, 2019
Updates the SDK's API error message SerializationError handling to
capture the original error message byte, and include it in the
SerializationError error value.

Also replaces the "SerializationError" literal string in the SDK with the constant, ErrCodeSerialization.

Fix aws#2562, aws#2411, aws#2315
diehlaws pushed a commit to diehlaws/aws-sdk-go that referenced this pull request Jul 2, 2019
Updates the pending change log with PR aws#2600.
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.

Kafka ListNodes SerializationError: failed decoding JSON RPC response
2 participants