ruby-grape / grape / 42788574468

100%

master: 100%

Pull #3076

github

Build the params of a body sent without a query string from the form params alone

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>

1615 of 1615 branches covered (100.0%)

Branch coverage included in aggregate %.

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

4704 of 4704 relevant lines covered (100.0%)

21400.5 hits per line