Security context

What an agent needs to avoid regressing past fixes and find the next vuln in this repo.

basecamp/google_sign_in
master @ e7f2a9a
4
Fixes
0
CVEs
CRITICAL
Peak severity
0.0%
Coverage
Highlights
Open Redirect: 3 prior fixes. Scrutinize any change in this area.
lib/google_sign_in/redirect_protector.rb: most-fixed (2 issues). Treat as high-risk during review.
3 high-severity fixes in this history; regressions here are high-impact.
Recurring patterns

The bug types that recur here, drawn from past fixes, not open vulnerabilities.

Open Redirect: Two separate fixes were needed to close bypasses in the same-origin check, indicating the validation logic is subtle and prone to regressions.
Open Redirect: The controller explicitly disabled Rails' default host validation, which allowed redirects to external hosts; this is a distinct entry point from the protector.
Auth Bypass: Missing audience validation allowed tokens with wrong client_id to be accepted, leading to full authentication bypass.