Skip to content

fix: add request context to JSON decode errors#275

Merged
BoxBoxJason merged 1 commit into
mainfrom
fix/decode-error-content
Jun 13, 2026
Merged

fix: add request context to JSON decode errors#275
BoxBoxJason merged 1 commit into
mainfrom
fix/decode-error-content

Conversation

@BoxBoxJason

Copy link
Copy Markdown
Owner

Description of your changes

This PR passes some context about the HTTP error when a decode error happens at runtime.
This helps the user understand what went wrong properly.

Fixes #252

I have:

  • Followed the git conventional commit message format.
  • Made sure all changes are covered by proper tests, reaching a coverage of at least 80% when applicable.

How has this code been tested

I have:

  • Made sure make lint passes to verify that the code style is correct.
  • Made sure make test passes to verify that the code is working as intended.
  • Made sure make e2e passes to verify that end-to-end tests pass against a real SonarQube instance.
  • Added unit tests to cover the code changes.
  • Added end-to-end tests if necessary.

Signed-off-by: BoxBoxJason <contact@boxboxjason.dev>
@BoxBoxJason BoxBoxJason added this to the User Production Readiness milestone Jun 13, 2026
@BoxBoxJason BoxBoxJason self-assigned this Jun 13, 2026
@BoxBoxJason BoxBoxJason added enhancement New feature or request unit-test Unit test related work sdk SonarQube golang SDK related work labels Jun 13, 2026
@BoxBoxJason BoxBoxJason merged commit de45efc into main Jun 13, 2026
4 checks passed
@BoxBoxJason BoxBoxJason deleted the fix/decode-error-content branch June 13, 2026 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request sdk SonarQube golang SDK related work unit-test Unit test related work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

JSON decode error from Do is returned without context

1 participant