Commit 287176b
Bump radiance to ca14a8c — fix x-lantern-client-country header leak (#8727)
Picks up radiance #456: revert config/fetcher.go's x-lantern-client-country
read source from settings.GetString(settings.CountryCodeKey) back to
env.GetString(env.Country). PR #370's LocalBackend refactor inadvertently
widened the header from a dev-only env knob to an auto-populated setting,
causing every production client to send the header on every /config-new
after their first response — which short-circuits server-side MaxMind ASN
lookup and breaks per-ASN bandit learning.
SigNoz showed ~21.6% of /config-new responses had bandit.asn=0 in the last
24h (~82% caused by this header), CN at ~100% impact, rate rising 6x
week-over-week. Refs engineering #3398.
Co-authored-by: Adam Fisk <afisk@mini.local>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>1 parent bcc8973 commit 287176b
2 files changed
Lines changed: 3 additions & 3 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
28 | | - | |
| 28 | + | |
29 | 29 | | |
30 | 30 | | |
31 | 31 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
261 | 261 | | |
262 | 262 | | |
263 | 263 | | |
264 | | - | |
265 | | - | |
| 264 | + | |
| 265 | + | |
266 | 266 | | |
267 | 267 | | |
268 | 268 | | |
| |||
0 commit comments