ruby-grape / grape / 30218032275

97%

master: 97%

Build Type

push

github

Committed by ericproulx

Commit Message

Let a cascading route hand over to every remaining route

Router#transaction sent a cascading match straight to the greedy
neighbour — the last route registered for the path — and returned its
answer, so any route registered between the first match and that last
one was unreachable. With three mounted API versions plus a catch-all
`route :any, '*path'`, v1 and v3 were served while v2 answered 406; the
shortcut only ever looked correct with exactly two versions, where "the
last route" happens to be the sibling.

Try the method's remaining routes (#rotation) as soon as a match
cascades, before the ANY routes and the greedy neighbour, and keep the
priority order otherwise intact. A consequence, and what cascade: true
means: with a catch-all present, an unmatched version now falls through
to it rather than surfacing the versioner's 406. cascade: false still
answers 406 immediately and never consults the catch-all.

Also close the body of every superseded cascading response — Rack
requires it, and rotation previously leaked them (Rack::Lint flagged
this as soon as a third sibling made the path reachable).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Coverage Stats

1175 of 1266 branches covered (92.81%)

Branch coverage included in aggregate %.

28 of 29 new or added lines in 1 file covered. (96.55%)

1 existing line in 1 file now uncovered.

3799 of 3866 relevant lines covered (98.27%)

25868.46 hits per line