stacklok / toolhive / 24363956980

66%

Build Type

push

github

Committed by web-flow

Commit Message

Support stateless streamable-HTTP remote servers (#4515)

* Support stateless streamable-HTTP remote servers

Remote MCP servers implementing the streamable-HTTP transport in
stateless mode (POST-only, no SSE) show zero tools when proxied by
ToolHive. The transparent proxy forwards all HTTP methods including
GET; stateless servers don't serve SSE on GET, so clients never
complete tool discovery.

Add a --stateless flag to thv run that gates GET, HEAD, and DELETE
requests at the proxy with 405, and switches the health check to a
POST-based JSON-RPC ping appropriate for servers without an SSE
endpoint.

Fixes #4514

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Add --stateless flag to generated docs

Regenerate CLI and Swagger docs to include the new --stateless
flag added to the run command.

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Apply stateless mode to all transports

Move stateless configuration outside the remote-URL guard so
--stateless works for both remote URLs and local container
workloads. Add Allow header to 405 responses per RFC 9110.
Update comments and flag text to remove remote-only language.

Signed-off-by: Greg Katz <gkatz@indeed.com>

* Add E2E test for stateless proxy mode

Verify that --stateless makes the proxy reject GET and DELETE
with 405 and forward POST to the upstream MCP server. Uses an
in-process mock server to avoid external dependencies.

Signed-off-by: Greg Katz <gkatz@indeed.com>

---------

Signed-off-by: Greg Katz <gkatz@indeed.com>

Coverage Stats

68 of 98 new or added lines in 6 files covered. (69.39%)

14 existing lines in 3 files now uncovered.

56854 of 86711 relevant lines covered (65.57%)

62.06 hits per line