ruby-grape / grape / 42704818184

100%

push

github

Skip the query parse for a request with no query string (#3071)

A route that reads its params -- any route declaring a param, a path
param included -- had Rack parse the query string to build them, the
empty one too: a parser call, two objects and a Hash memoized in the env
for a request that carries nothing to parse. A GET, PUT or DELETE by id
is the common case of it.

Such a request now hands the params builder a shared frozen Hash. A
builder that returns its argument is still copied, as it is for the Hash
Rack memoizes. `Rack::Request#GET` is untouched and parses on demand.

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

1599 of 1599 branches covered (100.0%)

Branch coverage included in aggregate %.

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

4686 of 4686 relevant lines covered (100.0%)

19598.61 hits per line