coverallsapp / coverage-reporter / 31825817418

95%

push

github

Stop relying on undocumented HTTP::Headers#to_json in debug log (#188)

The debug log line at jobs.cr:51 invoked `headers.to_pretty_json` on an
HTTP::Headers value, which depends on an undocumented Crystal stdlib
overload `HTTP::Headers#to_json(JSON::Builder)`. That overload vanished
in Crystal 1.20.0, breaking the source build for Homebrew users without
matching bottles, and was only restored in 1.20.1 as an explicit forwarder
the Crystal team has signaled is unintentional.

Convert HTTP::Headers to a plain Hash before serializing so we go through
documented JSON paths only. Survives any future Crystal stdlib churn
around HTTP::Headers serialization.

Adds a regression spec exercising send_request under Log::Level::Debug.

Closes #187
Refs https://github.com/coverallsapp/homebrew-coveralls/issues/71
Refs https://github.com/crystal-lang/crystal/issues/16886

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

1 of 1 new or added line in 1 file covered. (100.0%)

937 of 995 relevant lines covered (94.17%)

2.14 hits per line