umputun / reproxy / 25721842582

78%

master: 79%

Pull #251

github

fix: address Copilot review on PR #251

Apply modulo on lastSelected write to keep [0, n) invariant - matches
original code's wrap-to-0 semantics when n changes, while still bounding
the returned value to current n. Both forms fix the panic; this one has
smaller behavioral delta from pre-fix code.

Add TestRoundRobinSelector_SelectGrowingN that pins the wrap behavior
when n grows back (e.g., dead backend recovers), catching regression
to the non-modulo write.

Expand the //nolint:gosec comment on the http->https redirect to spell
out the threat model (Host header is browser-set from URL hostname,
so a foreign Host can't be injected into a victim's request).

5 of 5 new or added lines in 2 files covered. (100.0%)

2417 of 3084 relevant lines covered (78.37%)

24.5 hits per line