coverallsapp / coverage-reporter / 31849246696

95%

push

github

Show trace IDs in API error output (#185)

* WIP: capture and display trace IDs in API error output

Adds a `headers` getter and `trace_id` helper to Api::HTTPError. The
helper probes common CDN/cloud tracing headers (cf-ray, x-request-id,
x-amzn-requestid, x-amzn-trace-id, x-trace-id) and returns the first
match. The CLI's error rescues for InternalServerError, UnprocessableEntity
and the generic HTTPError now print "Trace ID: ..." alongside the
existing status/response so users can include it when contacting support.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

* Show trace IDs in API errors; drop 500 response-body dump

Finishes the trace-ID work:

- Suppress the `Trace ID:` line entirely when the response carried no
  tracing header, instead of emitting a blank line.
- Stop printing the response body for `InternalServerError`. A 500 is
  usually a CDN/proxy HTML error page, which is noise; the trace ID is
  the part support can act on.
- Add specs for `Api::HTTPError#trace_id`: every header in the probe
  chain, case-insensitive matching, precedence, fall-through, the nil
  cases, and both rescued subclasses.

Also resolves the codebase's only Warning-level ameba finding
(`Lint/UnusedRescueVariable` at cmd.cr:48).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

13 of 14 new or added lines in 2 files covered. (92.86%)

1025 of 1086 relevant lines covered (94.38%)

2.13 hits per line