ruby-grape / grape / 32583678355

97%

Build Type

push

github

Committed by web-flow

Commit Message

Let rescue_from :grape_exceptions outrank a catch-all class handler (#2843)

rescue_from :grape_exceptions is an opt-in to keep Grape's own errors
rendering with their own status -- a validation failure answers 400 rather
than whatever the application's catch-all returns.

It only ever worked against rescue_from :all, which lives in
all_rescue_handler and is consulted last. Written as a class instead,
rescue_from StandardError is a registered handler, matched first, and Grape's
exceptions are StandardErrors -- so the opt-in silently did nothing and
validation errors still came back as 500s:

    rescue_from StandardError { error!('server error', 500) }
    rescue_from :grape_exceptions          # inert

Let the opt-in win over a handler that only matched through a non-Grape
ancestor. A handler registered for a Grape exception class is more precise
than the opt-in and still wins, so an explicit
rescue_from Grape::Exceptions::ValidationErrors keeps its handler.

registered_rescue_handler gains an _entry variant returning the matched class
alongside its handler, since deciding this needs to know *which* class matched,
not just that one did. find_handler resolves the entry once and passes it down.

Application errors still reach the catch-all, rescue_from :all is unchanged,
and InvalidVersionHeader is left alone so it keeps reaching Rack and version
cascading still works.

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

Coverage Stats

1217 of 1330 branches covered (91.5%)

Branch coverage included in aggregate %.

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

43 existing lines in 9 files now uncovered.

3851 of 3962 relevant lines covered (97.2%)

5603.98 hits per line