Highlights
Auth Bypass: 12 prior fixes. Scrutinize any change in this area.
cmd/subscribers.go: most-fixed (6 issues). Treat as high-risk during review.
22 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 endpoints lacked RBAC checks, allowing unauthorized subscriber and bounce operations. The fix added hasSubPerm checks, but sibling handlers may still miss them.
Missing Authorization: A single commit fixed multiple missing authorization checks across campaign, import, and subscriber handlers, indicating a systemic gap in permission enforcement for sensitive operations like blocklisting, deleting, and exporting.
Privilege Escalation: The subscribers:sql_query permission was not enforced, allowing non-superadmin users to execute arbitrary SQL and join the sessions table to escalate privileges. The fix added a permission check and table allowlist.