ruby-grape / grape / 42795739763

100%

Build Type

push

github

Committed by web-flow

Commit Message

Build the params of a body sent without a query string from the form params alone (#3076)

Rack::Request#params is the query params merged with the form params
into a new Hash. A request with a body and no query string has nothing
to merge them with, yet Rack was still asked for both: it parsed the
empty query string, memoized the empty Hash that makes in the env, and
copied the form params into a third Hash for the builder to copy again.

Such a request is now built from the form params Rack memoized, as a
request with a query string and no body already is from its query
params (#3071). A request that sends both is merged as before.

Co-authored-by: Claude Fable 5.2 <noreply@anthropic.com>

Coverage Stats

1613 of 1615 branches covered (99.88%)

Branch coverage included in aggregate %.

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

8 existing lines in 2 files now uncovered.

4696 of 4704 relevant lines covered (99.83%)

8570.04 hits per line