Skip to content

Improve iotclient error details #62

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

Merged
merged 2 commits into from
Nov 18, 2021
Merged

Improve iotclient error details #62

merged 2 commits into from
Nov 18, 2021

Conversation

polldo
Copy link
Contributor

@polldo polldo commented Nov 17, 2021

Motivation

Some errors are returned without an err model. In such cases, the details of the response are in the error body.

Change description

Additional Notes

Reviewer checklist

  • PR address a single concern.
  • PR title and description are properly filled.
  • Changes will be merged in main.
  • Changes are covered by tests.
  • Logging is meaningful in case of troubleshooting.
  • History is clean, commit messages are meaningful (see CONTRIBUTING.md) and are well formatted.

Some errors are returned without an err model. In such cases, the details of the response are in the error body.
@polldo polldo force-pushed the polldo/errors-details branch from 3c7531f to 08ba83a Compare November 18, 2021 11:17
@polldo polldo changed the title Improve iotclient errors details Improve iotclient error details Nov 18, 2021
@polldo polldo merged commit 975a828 into main Nov 18, 2021
@polldo polldo deleted the polldo/errors-details branch November 18, 2021 11:23
polldo pushed a commit that referenced this pull request Sep 2, 2022
Some errors are returned without an err model. In such cases, the details of the response are in the error body.
polldo pushed a commit that referenced this pull request Sep 2, 2022
Some errors are returned without an err model. In such cases, the details of the response are in the error body.
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.

1 participant