ruby-grape / grape / 40224778427

100%

push

github

Wrap the router in Rack::Head instead of each endpoint's stack (#3042)

Rack::Head was the first middleware in each endpoint's stack, so it only
saw responses that went through one. A Rack app mounted with `mount` is
called by the router directly and skips `build_stack`, and a middleware
inserted with `insert 0` sat outside Rack::Head: a HEAD request to
either got the full body back. The router's own 404 had to strip its
body itself (#2958) for the same reason.

A single Rack::Head around the router, inside Rack::Lint, now strips
the body of every HEAD response the API gives, and the router's 404
special case goes away. A request that cascades through several routes
goes through one Rack::Head instead of one per route tried.

Since an endpoint's stack no longer contains Rack::Head,
`insert_before`/`insert_after Rack::Head` raise when the API compiles,
and a middleware inserted by index lands one place further in. Both are
covered in UPGRADING.

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

1519 of 1521 branches covered (99.87%)

Branch coverage included in aggregate %.

7 of 7 new or added lines in 2 files covered. (100.0%)

4537 of 4538 relevant lines covered (99.98%)

30414.45 hits per line