cache: Key by depot path across CDN host aliases #48

Merged
linus merged 1 commit from moss/cache-key-depot-path into develop 2026-09-07 20:18:26 +00:00

Why

Steam rotates CDN hostnames for the same depot path. Keying from r.URL.String() could embed the Host (absolute-form targets) and miss-stampede across aliases.

What

  • Cache key from r.URL.Path only (never Host)
  • Test: same depot path under cdn1/cdn2 Hosts and absolute-form cdn3 shares one entry / one upstream fetch
  • README: one operator line on path-based keying

Test

go test ./steamcache/ -count=1

Rollback

git revert

Closes #39

## Why Steam rotates CDN hostnames for the same depot path. Keying from `r.URL.String()` could embed the Host (absolute-form targets) and miss-stampede across aliases. ## What - Cache key from `r.URL.Path` only (never Host) - Test: same depot path under cdn1/cdn2 Hosts and absolute-form cdn3 shares one entry / one upstream fetch - README: one operator line on path-based keying ## Test `go test ./steamcache/ -count=1` ## Rollback `git revert` Closes https://git.s1d3sw1ped.com/s1d3sw1ped/steamcache2/issues/39
cache: Key by depot path across CDN host aliases
All checks were successful
CI / vulncheck (pull_request) Successful in 14s
CI / check-and-test (pull_request) Successful in 41s
337741e06e
Ghost requested review from eva 2026-09-07 20:16:44 +00:00
eva was assigned by Ghost 2026-09-07 20:16:44 +00:00
linus merged commit 8eb31143e5 into develop 2026-09-07 20:18:26 +00:00
linus deleted branch moss/cache-key-depot-path 2026-09-07 20:18:26 +00:00
Sign in to join this conversation.
No description provided.