stacklok / toolhive / 22948654878

push

github

Fix deterministic E2E proxy SSE endpoint rewrite tests (#4093)

Fix deterministic E2E proxy SSE endpoint rewrite test failures

These SSE endpoint rewrite tests were orphaned from CI until the matrix
rebalance (#4074), then had stale flags fixed (#4080), but have never
actually passed. Four root causes are addressed:

1. GenerateMCPServerURL omits /sse for remote SSE URLs with empty path,
   causing waitForInitializeSuccess to GET / instead of /sse (404).
2. getSSERewriteConfig reads X-Forwarded-* from the outbound request
   which has auto-injected headers from SetXForwarded(), converting
   relative endpoint URLs into absolute ones with the proxy's own host.
3. Test 3 mock server lacks a 404 fallback for non-/sse paths.
4. Test 4 (OSV + endpoint-prefix) fails due to #3372; skip it.

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

30 of 30 new or added lines in 3 files covered. (100.0%)

47610 of 74452 relevant lines covered (63.95%)

73.82 hits per line