Security context

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

elazarl/goproxy
master @ 636207d
9
Fixes
1
CVEs
HIGH
Peak severity
Highlights
Auth Bypass: 4 prior fixes. Scrutinize any change in this area.
certs.go: 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.

Auth Bypass: Overwriting headers instead of appending them in copyHeaders when keepDestHeaders is active can lead to the loss of critical security or authentication headers (such as Set-Cookie), resulting in downstream authentication bypasses.
CWE-295: The default proxy client configuration previously initialized with InsecureSkipVerify: true, which completely disabled TLS certificate verification and allowed arbitrary man-in-the-middle attacks.
Server-Side Request Forgery: An unanchored regular expression check for the 'https://' prefix during HTTPS MITM handling allowed attackers to manipulate the URL structure, leading to Server-Side Request Forgery (SSRF) and routing bypasses.