stacklok / toolhive / 24227816235

66%

Build Type

push

github

Committed by web-flow

Commit Message

Fix health-check close failing auth in vMCP BackendClient (#4613)

* Fix health-check close failing auth in vMCP BackendClient

mcp-go's StreamableHTTP.Close() creates a DELETE request with
context.Background(), discarding the health-check marker and identity
from the original call context. The identityPropagatingRoundTripper
only stored identity, so when ListCapabilities deferred c.Close(),
the DELETE hit auth strategies with neither a health-check marker nor
an identity, producing "no identity found in context" errors.

Fix by capturing isHealthCheck at transport creation time and
re-injecting both identity and the health-check marker into every
outgoing request, including the synthetic DELETE from Close().

Fixes #4573

* fixes from review

---------

Co-authored-by: taskbot <taskbot@users.noreply.github.com>

Coverage Stats

28 of 28 new or added lines in 5 files covered. (100.0%)

36 existing lines in 4 files now uncovered.

56830 of 87217 relevant lines covered (65.16%)

62.47 hits per line