stripe / smokescreen / 24177641349

56%

master: 56%

Build Type

Pull #286

github

Committed by harold-s

Commit Message

smokescreen: remove TestHostSquareBrackets

Go 1.21 tightened URL validation in both the HTTP client (url.Parse) and
the HTTP server (request-line parsing), causing both layers to reject
hostnames containing square brackets before the request reaches
smokescreen's handler. The test can no longer exercise smokescreen's own
host-validation logic for these inputs and is removed.

Committed-By-Agent: claude

Pull Request Pull Request #286: acl: pass *http.Request through Decide via DecideArgs

Coverage Stats

14 of 15 new or added lines in 2 files covered. (93.33%)

9 existing lines in 1 file now uncovered.

1812 of 3224 relevant lines covered (56.2%)

46.51 hits per line