Skip to content

refactor(transports): remove SSE server mode#558

Merged
droot merged 2 commits into
GoogleCloudPlatform:mainfrom
appleboy:sse
Oct 23, 2025
Merged

refactor(transports): remove SSE server mode#558
droot merged 2 commits into
GoogleCloudPlatform:mainfrom
appleboy:sse

Conversation

@appleboy
Copy link
Copy Markdown
Contributor

@appleboy appleboy commented Oct 4, 2025

  • Remove support and documentation for MCP server SSE mode
  • Update descriptions and flag help messages to reflect only supported stdio and streamable-http modes
  • Clarify that HTTP port is only used with streamable-http mode
  • Remove code paths related to SSE server startup

This replaces the HTTP+SSE transport from protocol version 2024-11-05.

In the Streamable HTTP transport, the server operates as an independent process that can handle multiple client connections. This transport uses HTTP POST and GET requests. Server can optionally make use of Server-Sent Events (SSE) to stream multiple server messages. This permits basic MCP servers, as well as more feature-rich servers supporting streaming and server-to-client notifications and requests.

reference: https://modelcontextprotocol.io/specification/2025-06-18/basic/transports#streamable-http

…tion

- Remove support and documentation for MCP server SSE mode
- Update descriptions and flag help messages to reflect only supported stdio and streamable-http modes
- Clarify that HTTP port is only used with streamable-http mode
- Remove code paths related to SSE server startup

Signed-off-by: appleboy <appleboy.tw@gmail.com>
@droot droot requested a review from ShubyM October 6, 2025 21:05
@appleboy
Copy link
Copy Markdown
Contributor Author

@droot @ShubyM Any feedback?

@ShubyM
Copy link
Copy Markdown
Collaborator

ShubyM commented Oct 20, 2025

The change looks good! But I'm not sure if we want to drop SSE mode entirely, @droot if we are okay with dropping it, this looks good to merge!

@appleboy
Copy link
Copy Markdown
Contributor Author

@ShubyM The latest protocol 2025-06-18 currently defines two standard transport mechanisms for client-server communication:

@appleboy
Copy link
Copy Markdown
Contributor Author

@droot Any feedback for the PR?

@droot
Copy link
Copy Markdown
Member

droot commented Oct 22, 2025

The change looks good! But I'm not sure if we want to drop SSE mode entirely, @droot if we are okay with dropping it, this looks good to merge!

Given that latest specification of the protocol has removed sse, its safe to take it out.

@appleboy
I have approved the PR, but there is conflict on the docs file, once addressed, I can get this in.

@appleboy
Copy link
Copy Markdown
Contributor Author

appleboy commented Oct 23, 2025

@droot Resolved the conflicts. 315a1a2

@droot droot merged commit e5a5d6e into GoogleCloudPlatform:main Oct 23, 2025
7 checks passed
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.

3 participants