Highlights
Auth Bypass: 16 prior fixes. Scrutinize any change in this area.
internal/auth/auth.go: most-fixed (7 issues). Treat as high-risk during review.
19 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: Critical credential checking logic was severely broken by comparing input passwords against usernames or raw inputs instead of their stored hashes. Developers must ensure correct parameter order and slice references when executing bcrypt comparisons.
Auth Bypass: Authorization rules routinely failed open or evaluated inaccurately on nil contexts, parsing errors, or container label partial matches. Applying explicit deny-by-default behavior and exact matching is critical to maintaining boundary isolation.
Denial of Service: A lack of resource limits on login attempt tracking and session cleanup queues made the application highly vulnerable to memory exhaustion and lockouts. Enforcing maximum queue sizes and strict resource caps on in-memory caches is mandatory.