Fix route processing for AppSec#5987
Conversation
🎉 All green!🧪 All tests passed 🎯 Code Coverage (details) 🔗 Commit SHA: c12beb0 | Docs | Datadog PR Page | Give us feedback! |
Strech
left a comment
There was a problem hiding this comment.
Thanks for the fix, I left few minor suggestions
BenchmarksBenchmark execution time: 2026-07-03 09:20:50 Comparing candidate commit c12beb0 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 48 metrics, 1 unstable metrics.
|
What does this PR do?
Motivation:
When Warden invokes
Devise::FailureAppdirectly, the router is bypassed soActionDispatch::Routing::RouteSetnever runs andpath_parametersisnilinenv. This causes aNoMethodError.Change log entry
Yes. AppSec: Fix an exception for unauthorized requests when using Devise with Rails.
Additional Notes:
SCRS-2273
How to test the change?
CI.