{"id":"GHSA-8fcf-v89g-xpg6","summary":"Traefik: BasicAuth singleflight coalescing reintroduces an unauthenticated username-enumeration timing oracle","details":"## Summary\n\nTraefik's BasicAuth middleware coalesces concurrent credential checks through a `singleflight.Group` to avoid hashing the same password many times at once. Since v3.6.11 the deduplication key was built from the submitted password plus the stored secret, so it depended on server state: a non-existent username collapsed onto one shared key while each configured username produced its own. Under attacker-controlled concurrency, a probe request arriving inside a leader request's in-flight window is served the leader's fast coalesced result when the username does not exist, but computes its own hash (slow) when the username exists — reintroducing, only in the concurrent case, the unauthenticated username-enumeration timing oracle that GHSA-g3hg-j4jv-cwfr had hardened for sequential probing. The fix derives the singleflight key from the submitted credentials (username and password) only, so it no longer depends on whether the account exists or on any stored secret. Traefik v2 is not affected: the v2.11 BasicAuth middleware does not use singleflight coalescing, and Digest authentication is not affected. The impact is limited to username enumeration; no credential disclosure or authentication bypass is possible.\n\nThe vulnerability originates on the v3.6 line, which has reached end of life; users on v3.6 or earlier v3.x must upgrade to v3.7.13 to receive the fix.\n\n## Patches\n\n- https://github.com/traefik/traefik/releases/tag/v3.7.13\n\n## For more information\n\nIf you have any questions or comments about this advisory, please [open an issue](https://github.com/traefik/traefik/issues).\n\n\u003cdetails\u003e\n\u003csummary\u003eOriginal Description\u003c/summary\u003e\n\n## Summary\n\nConfirmed. `checkPassword` derives the `singleflight` key from the *stored secret*, so the key encodes whether the submitted username exists:\n\n- username absent, `secret == \"\"`, key `= len(P) + \":\" + P`\n- username present, key `= len(P) + \":\" + P + secret_T`\n\nEvery non-existent username therefore lands on one shared key, while every configured username gets its own. `singleflight.Group.Do` makes a follower on an equal key block on the leader's in-flight computation and return the leader's result. So an attacker who sends a leader request with a junk username and password `P`, then sends the probe for target `T` with the same `P` late inside the leader's window, reads user existence directly off the probe's latency: coalesced (fast) means `T` does not exist, own hash (slow) means `T` does exist.\n\nThis is the exact information leak that the `notFoundSecret` dummy hash at line 127 exists to remove. Sequential probing is fully equalised (measured ratio 1.00x, so the fix for `cwfr` / `8j2h` does work); the leak reappears only under attacker-controlled concurrency. Confirmed on the current `v3.7` head, which already carries the `8mrf` singleflight fix, and on `master` (no later fix exists).\n\n## Affected code\n\n- `pkg/middlewares/auth/basic_auth.go:125` (`checkPassword`)\n\n## Code analysis\n\n`pkg/middlewares/auth/basic_auth.go:119-131` matches the finding verbatim, including the cited line 125:\n\n```go\nfunc (b *basicAuth) checkPassword(user, password string) bool {   // :119\n\tsecret := b.auth.Secrets(user, b.auth.Realm)                   // \"\" when the user is absent\n\n\tkey := strconv.Itoa(len(password)) + \":\" + password + secret   // :124\n\tmatch, _, _ := b.singleflightGroup.Do(key, func() (any, error) {   // :125\n\t\tif secret == \"\" {\n\t\t\t_ = b.checkSecret(password, b.notFoundSecret)          // :127  dummy hash, equal cost\n\t\t\treturn false, nil\n\t\t}\n\t\treturn b.checkSecret(password, secret), nil\n\t})\n\treturn match.(bool)\n}\n```\n\nTwo independent properties combine:\n\n1. **The dummy hash equalises the cost of one lookup.** `notFoundSecret` is a configured user's real hash (`slices.Collect(maps.Values(users))[0]`), so absent and present users each perform exactly one hash of the same algorithm and cost. That is why the sequential control below is flat.\n2. **The key partitions on existence, so coalescing is not equalised.** The dummy hash was placed *inside* the closure (commit `122175ac2`, PR #12803), which is what pulls absent users into `Do` at all. Before that refactor, `secret == \"\"` returned `false` before `Do` was ever called.\n\nGit archaeology of the whole sequence in this one function:\n\n| Commit | Date | Effect |\n|---|---|---|\n| `6f469ee1e` | 2024-10-10 | Introduces `singleflight` to dedupe concurrent hashes (`Only calculate basic auth hashes once for concurrent requests`). Absent users returned `false` before `Do`. |\n| `122175ac2` (PR #12803) | 2026-03-17 | Fix for `cwfr`. Moves the empty-secret branch inside the closure, which makes the key existence-dependent for the first time. |\n| `8c4fc8957` | 2026-04-13 | Fix for `8j2h`: `notFoundSecret` was resolving to `\"\"`, so the dummy hash was a no-op. |\n| `b5ace8eb5` (PR #13572) | 2026-07-28 | Fix for `8mrf`: adds the `len(password) + \":\"` prefix so distinct (password, secret) pairs cannot alias. Keeps the secret in the key and keeps the empty-secret branch inside the closure. |\n\nJ15 is the residue of that last fix. It does **not** depend on the key collision `8mrf` closed: the oracle works precisely *because* the keys differ. The prior analysis of `8mrf` recommended keeping \"the empty-secret (unconfigured) path out of any key that a configured user can share\"; the shipped patch only delimited the key, so the existence dependency survived.\n\nAffected range: `\u003e= v3.6.11` (the `122175ac2` refactor) through current `v3.7` head and `master`. Not the v2 line, and not earlier v3 releases, which returned early for absent users. Digest auth does not use `singleflight` and is unaffected.\n\nImpact scope: username enumeration only. No credential disclosure, no authentication bypass, no result sharing across identities.\n\n## Reproduction\n\nGo tests written directly in `package auth`, driving the real `NewBasic` handler over a real HTTP server (`httptest`), with real `bcrypt` / `apr1` hashes and no instrumentation of the vulnerable logic. Files: `pkg/middlewares/auth/zz_scanpoc_J15{,b,c}_test.go`, deleted after the run.\n\n**Commands**\n\n```\ncd /Users/emile/go/src/github.com/traefik/traefik\ngo test -count=1 -run TestZZScanPoCJ15      -v ./pkg/middlewares/auth/...\ngo test -count=1 -run TestZZScanPoCJ15Costs -v ./pkg/middlewares/auth/...\ngo test -count=1 -run TestZZScanPoCJ15H2    -v ./pkg/middlewares/auth/...\n```\n\nProbe shape, exactly the claimed scenario: calibrate `D` with one request, launch a leader with a junk username and password `P`, sleep `0.9 * D`, then send the probe with password `P` and either a configured (`alice`) or an absent (`bob`) username, and time the probe. `classify=OK` means `present \u003e 2 * absent`, i.e. the oracle answered correctly.\n\n**Observed, main PoC (test 1)**\n\n```\n[bcrypt-cost12] SEQUENTIAL control: absent=249.568275ms present=249.10215ms ratio=1.00x\n[bcrypt-cost12] CONCURRENT D=245.521625ms frac=0.90 round 0: absent=21.750833ms present=242.081792ms ratio=11.1x  classify=OK\n[bcrypt-cost12] CONCURRENT D=245.521625ms frac=0.90 round 1: absent=20.197667ms present=245.234958ms ratio=12.1x  classify=OK\n[bcrypt-cost12] CONCURRENT D=245.521625ms frac=0.90 round 2: absent=25.675958ms present=243.981375ms ratio=9.5x   classify=OK\n[bcrypt-cost10] SEQUENTIAL control: absent=61.4374ms present=61.049608ms ratio=0.99x\n[bcrypt-cost10] CONCURRENT D=60.54675ms frac=0.90 round 0: absent=5.572875ms present=60.8345ms   ratio=10.9x  classify=OK\n[bcrypt-cost10] CONCURRENT D=60.54675ms frac=0.90 round 1: absent=4.042292ms present=61.988792ms ratio=15.3x  classify=OK\n[bcrypt-cost10] CONCURRENT D=60.54675ms frac=0.90 round 2: absent=4.53825ms  present=61.557791ms ratio=13.6x  classify=OK\n[apr1-short-pw] SEQUENTIAL control: absent=360.7µs present=373.116µs ratio=1.03x\n[apr1-short-pw] CONCURRENT D=370.792µs frac=0.90 round 0: absent=378.167µs present=409.458µs ratio=1.1x  classify=FAIL\n[apr1-short-pw] CONCURRENT D=370.792µs frac=0.90 round 1: absent=352.416µs present=333.875µs ratio=0.9x  classify=FAIL\n[apr1-short-pw] CONCURRENT D=370.792µs frac=0.90 round 2: absent=374.209µs present=349.125µs ratio=0.9x  classify=FAIL\n[apr1-8000B-pw] SEQUENTIAL control: absent=22.159325ms present=21.999433ms ratio=0.99x\n[apr1-8000B-pw] CONCURRENT D=22.457583ms frac=0.90 round 0: absent=22.089458ms present=22.516708ms ratio=1.0x   classify=FAIL\n[apr1-8000B-pw] CONCURRENT D=22.457583ms frac=0.90 round 1: absent=698.042µs  present=21.877ms    ratio=31.3x  classify=OK\n[apr1-8000B-pw] CONCURRENT D=22.457583ms frac=0.90 round 2: absent=1.570875ms present=21.945417ms ratio=14.0x  classify=OK\n```\n\nThe **sequential control is the decisive part**: 1.00x / 0.99x / 1.03x / 0.99x on every algorithm. The constant-time countermeasure is intact for one-request-at-a-time probing, so the 10x to 15x concurrent separation is attributable to the coalescing and to nothing else. That rules out the alternative explanation that this is just `cwfr` / `8j2h` still unfixed.\n\n**Observed, per-algorithm hash cost (test 2)**\n\n```\nbcrypt cost10 / short pw        -\u003e 60.5919ms per hash\nbcrypt cost12 / short pw        -\u003e 242.384558ms per hash\napr1 / short pw                 -\u003e 128.333µs per hash\napr1 / 8000-byte pw             -\u003e 42.575816ms per hash\n```\n\n**Observed, single HTTP/2 connection (test 3)**\n\nBoth probes multiplexed as two streams over one TLS connection, which pins them to a single Traefik process even behind an L4 load balancer fronting several replicas:\n\n```\nh2 single-connection: D=63.048ms\nh2 round 0: absent=7.931459ms present=65.87375ms  ratio=8.3x  classify=OK\nh2 round 1: absent=4.255667ms present=64.193291ms ratio=15.1x classify=OK\nh2 round 2: absent=6.696417ms present=64.695208ms ratio=9.7x  classify=OK\n```\n\n**Conclusion: REPRODUCED**, 9/9 correct classifications on bcrypt, single-shot, no statistics.\n\nClaim-by-claim audit of the finding text:\n\n_(truncated ; full analysis in the linked internal report)_\n\n## Documentation grounding\n\n**Not working-as-intended. The governing project document puts this class explicitly in scope.**\n\n- `docs/content/security/` (header-underscores, request-path, content-length, http2-header-memory, multi-tenant-kubernetes) has no page covering BasicAuth, the timing posture or the singleflight dedup. Grep for `timing|basicauth|basic auth|enumerat|singleflight` across that directory: no match. No governing security doc, hence no WAI signal from there.\n- `docs/content/contributing/security-decisions.md`, section **Authentication Middleware Correctness**, is the settled public position and it is directly on point:\n\n  \u003e **Our position.** In scope: credential or identity handling that leaks across requests or users, **observable timing differences that disclose whether a principal exists**, and credentials forwarded to a destination the operator did not authorise.\n  \u003e\n  \u003e **Where the line is.** Choosing a weak authentication mechanism, or configuring it permissively, is the operator's decision. **The middleware failing to deliver what its documentation promises is ours.**\n\n_(truncated ; full analysis in the linked internal report)_\n\n## Precedent in comparable projects\n\nCorpus refreshed 2026-08-24 for nginx, ingress-nginx, kong, caddy, apisix, nginx-plus; envoy (2026-06-29), envoy-gateway (2026-06-15), haproxy (2026-07-16) and istio (2026-05-12) are staler.\n\nNo exact analogue of a request-deduplication timing oracle. Adjacent prior art:\n\n- **Envoy, CVE-2026-47775, medium**: \"OAuth2 Filter: Padding Oracle via AES-256-CBC Cookie Decryption\". A side-channel oracle inside an auth filter, published as a medium CVE. Framed on the observability of the discrepancy, not on the difficulty of measuring it.\n- **Envoy Gateway, CVE-2026-53715 / GHSA-8fv2-88gg-hm7q, medium**: \"Wasm cache ServeHTTP reads mappingPath2Cache without lock\". A concurrency defect in a shared per-process cache in the request path, treated as a real medium CVE and fixed by correcting the shared-state handling.\n- **APISIX, CVE-2025-62232, high**: basic-auth credential exposure. Same component, unrelated mechanism (logging).\n\nTakeaway: the industry treats side-channel oracles in auth filters, and concurrency defects in shared per-process request-path caches, as genuine publishable CVEs of roughly this severity. Nothing in the corpus argues the class is by design. The strongest precedent, however, is not a competitor: it is Traefik's own two published CVEs on this exact guarantee.\n\n## Recommended fix\n\nAssign for fix. No fix exists: `gh search prs --repo traefik/traefik \"singleflight\"` returns only PR #13572 (the merged `8mrf` key-collision fix) and `\"basic auth timing\"` only PR #12803 / #12796; `git log --all` on the checkout shows `b5ace8eb5` as the last change to `pkg/middlewares/auth/` and `master` (`174e5d811`, a merge of `v3.7`) carries nothing later.\n\n**Fix shape: remove the secret from the key and qualify it by username.** Something along the lines of\n\n```go\nkey := strconv.Itoa(len(user)) + \":\" + user + \":\" + password\n```\n\nwith the dummy-hash branch left inside the closure. This is strictly better than the current key on all four counts:\n\n- **It closes J15**: the key no longer encodes existence, so two absent users no longer share a bucket that a present user is excluded from. Coalescing then happens only for an identical `(user, password)` pair, which leaks nothing an attacker did not already supply.\n- **It closes `8mrf` structurally** rather than by delimiting: the stored hash never enters the key, so no choice of password can alias a configured user's key. The `len` prefix is still needed, now on `user`, to keep `(\"ab\", \"c\")` and `(\"a\", \"bc\")` apart.\n- **It preserves the purpose of `6f469ee1e`**: the case that commit exists for is a burst of concurrent requests carrying the *same* credentials, which is exactly what a username-qualified key still dedupes.\n- **It keeps the constant-time property**: one hash of `notFoundSecret` for absent users, one hash of `secret` for present ones, unchanged.\n\nDo not fix this by making the dummy branch share the leader's timing envelope; as the finding correctly notes, that only helps if the key stops depending on `secret == \"\"`.\n\nSecondary, low cost: restore the \"Timing attacks\" admonition that PR #12803 added to the BasicAuth page. It is the statement of the guarantee, it is the thing `security-decisions.md` holds the project to, and it is currently absent from the `v3.7` docs tree.\n\nIf filed: new cluster slug `basicauth-singleflight-existence-oracle`, sibling of `basicauth-singleflight-key-collision`, affected range `\u003e= v3.6.11` through the current `v3.6` / `v3.7` heads, v2 unaffected, digest auth unaffected. Correct the 40x claim and the `$apr1$` \"trivial\" claim in the published description per the audit table above.\n\n## Provenance\n\nFound by an external automated code scan (`CLAUDE-SECURITY-20260824-122205`) of `pkg/middlewares`, `pkg/proxy`, `pkg/server`, `pkg/muxer` and `pkg/tls` on branch `v3.7` at commit `d5072ce7b8765c9574246072e05dd81d84950da7`, then triaged with the `advisory-check` process : mechanism-level duplicate check against the existing advisory corpus, CVE-policy gate, security-documentation grounding, comparable-project precedent, and a mandatory reproduction attempt.\n\nTriage outcome : **Likely Valid**, confidence High, reproduced (yes). Expected publication likelihood at triage time : High.\n\nScanner finding id : F16. Internal report : `findings/scan-20260824/verdicts/J15.md` in the security-advisor repository.\n\n\u003c/details\u003e\n\n---","aliases":["CVE-2026-88010"],"modified":"2026-09-22T21:00:03.636567854Z","published":"2026-09-22T20:40:49Z","database_specific":{"github_reviewed":true,"github_reviewed_at":"2026-09-22T20:40:49Z","nvd_published_at":"2026-09-22T16:18:06Z","cwe_ids":["CWE-208"],"severity":"MODERATE"},"references":[{"type":"WEB","url":"https://github.com/traefik/traefik/security/advisories/GHSA-8fcf-v89g-xpg6"},{"type":"ADVISORY","url":"https://nvd.nist.gov/vuln/detail/CVE-2026-88010"},{"type":"WEB","url":"https://github.com/traefik/traefik/pull/13816"},{"type":"WEB","url":"https://github.com/traefik/traefik/commit/ddc1bf4660b85fd61fafdd821eb8216fb1a0b130"},{"type":"PACKAGE","url":"https://github.com/traefik/traefik"},{"type":"WEB","url":"https://github.com/traefik/traefik/releases/tag/v3.7.13"}],"affected":[{"package":{"name":"Traefik","ecosystem":"Go","purl":"pkg:golang/Traefik"},"ranges":[{"type":"SEMVER","events":[{"introduced":"3.6.11"},{"fixed":"3.7.13"}]}],"database_specific":{"last_known_affected_version_range":"\u003c= 3.7.12","source":"https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2026/09/GHSA-8fcf-v89g-xpg6/GHSA-8fcf-v89g-xpg6.json"}}],"schema_version":"1.9.0","severity":[{"type":"CVSS_V4","score":"CVSS:4.0/AV:N/AC:L/AT:P/PR:N/UI:N/VC:L/VI:N/VA:N/SC:N/SI:N/SA:N"}]}