iotaledger / iota-rust-sdk / 26793578453

36%

Build Type

push

github

Committed by web-flow

Commit Message

fix: surface HTTP status and body when GraphQL response decode fails (#1185)

* fix: surface HTTP status and body when GraphQL response decode fails

run_query and run_query_from_json used reqwest's Response::json(), which
discarded the HTTP status and body on a parse failure, leaving callers with
an opaque "expected value at line 1 column 1" decode error.

Both methods now read the response via a shared post_query helper that checks
the status and decodes the bytes explicitly. Non-2xx responses and JSON decode
failures are reported with the request URL, HTTP status (code + reason) and a
truncated, UTF-8-lossy body via the new Error::http / Error::decode
constructors (and a new Kind::Http variant).

Closes #1182

* fix: avoid duplicating error message in cause chain

Error::source() returned the inner error directly, whose own message is
already rendered inline by Display. Cause-chain formatters (anyhow/eyre
`{:?}`) therefore printed that message twice — once in the Display line and
again under "Caused by".

Expose only the underlying cause from source() so the message appears exactly
once while deeper sub-causes remain in the chain. Display (used by the FFI
bindings via to_string()) is unchanged.

* remove extra type qualifier

---------

Co-authored-by: Claude <noreply@anthropic.com>
Co-authored-by: Thibault Martinez <thibault@iota.org>

Coverage Stats

85 of 87 new or added lines in 2 files covered. (97.7%)

18 existing lines in 4 files now uncovered.

8852 of 22514 relevant lines covered (39.32%)

9560.85 hits per line