24006778672
97%
master: 97%

Ran 05 Apr 2026 05:34PM UTC

Jobs 0

Files 0

Run time –

Badge

Embed ▾

pending completion

cancel

Build # 24006778672

Build Type

push

github

Committed by

dblock

Commit Message

Fix UnknownAuthStrategy raised when custom auth strategy inherits from Grape::Middleware::Auth::Base

When a custom auth strategy class inherits from Grape::Middleware::Auth::Base
and is registered via Grape::Middleware::Auth::Strategies.add, it would raise
Grape::Exceptions::UnknownAuthStrategy with a blank strategy name upon being
instantiated by StrategyInfo#create.

The root cause: PR #2563 moved the strategy lookup from request-time (_call)
to initialization-time (initialize). When StrategyInfo#create instantiates
the custom class via auth_class.new(app) (without options), the inherited
initialize tries to look up options[:type] which is nil, causing the error.

The fix: guard the strategy lookup so it only runs when :type is present in
options. This preserves compile-time validation for the outer middleware
wrapper while allowing subclasses to be instantiated as actual strategies.

Fixes #2669.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Coverage Stats

Source Files on build 24006778672

Detailed source file information is not available for this build.

dblock / grape / 24006778672
97%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 24006778672

dblock / grape / 24006778672 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on build 24006778672

dblock / grape / 24006778672
97%
master: 97%

README BADGES
x