Security context

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

pglombardo/PasswordPusher

              master @ 7587363
            

Fixes

CVEs

CRITICAL

Peak severity

72.7%

Coverage

Highlights

Auth Bypass: 8 prior fixes. Scrutinize any change in this area.

app/controllers/passwords_controller.rb: most-fixed (6 issues). Treat as high-risk during review.

18 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.

Auth Bypass: Multiple fixes address missing or inconsistent ownership/authentication checks on push operations, including nil==nil bypass and unauthenticated deletion. Centralized owned_by? and deletable_by? are critical to prevent unauthorized access.

CSP Bypass: Repeated CSP hardening removed unsafe-inline/unsafe-eval, added nonces, and fixed predictable nonce generation. Weak CSP allows script injection and XSS.

Information Disclosure: Secrets (payload, passphrase, note) leaked via logs, JSON responses, and URL existence. Filtering and response sanitization are essential.