Skip to content

fix(proxy): scope fallback cache by target URL - #5261

Merged
diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.40from
KooshaPari:fix/proxy-fallback-cache-key
Jun 28, 2026
Merged

diegosouzapw merged 1 commit into
diegosouzapw:release/v3.8.40from
KooshaPari:fix/proxy-fallback-cache-key

Conversation

@KooshaPari

Copy link
Copy Markdown
Contributor

Proxy auto-fallback cached positive and negative probe results by hostname only. A failed probe for one endpoint on a shared API host could suppress a later probe for a different endpoint on that same host for the full fallback TTL.

This scopes fallback cache entries by normalized target URL while preserving the existing host fallback for malformed target URLs.

Validation:

  • env DISABLE_SQLITE_AUTO_BACKUP=true node --max-old-space-size=8192 --import tsx --test --test-force-exit tests/unit/proxy-fallback-cache-key.test.ts tests/unit/proxy-autoselect-optin-3332.test.ts -> 3/3 pass
  • npm run check:file-size -> pass
  • npx eslint open-sse/utils/proxyFallback.ts tests/unit/proxy-fallback-cache-key.test.ts tests/unit/proxy-autoselect-optin-3332.test.ts -> pass

@KooshaPari
KooshaPari requested a review from diegosouzapw as a code owner June 28, 2026 21:50
@gemini-code-assist

Copy link
Copy Markdown
Contributor

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@diegosouzapw
diegosouzapw merged commit 8adb40e into diegosouzapw:release/v3.8.40 Jun 28, 2026
7 checks passed
@diegosouzapw diegosouzapw mentioned this pull request Jun 29, 2026
@KooshaPari
KooshaPari deleted the fix/proxy-fallback-cache-key branch July 2, 2026 22:10
tkgo11 pushed a commit to tkgo11/OmniRoute that referenced this pull request Sep 23, 2026
Integrated into release/v3.8.40 — proxy fallback cache scoped by target URL (prevents cross-endpoint poisoning).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants