Skip to content

Conversation

@JM-Lab
Copy link
Contributor

@JM-Lab JM-Lab commented Jul 10, 2025

This pull request updates the default sse-message-endpoint in the Spring AI MCP server from /mcp/message to /mcp/messages, aligning it with the official MCP specification and the common practice used by MCP Framework (which defaults to /messages) to improve interoperability and avoid confusion.

@markpollack markpollack added this to the 1.1.0.M4 milestone Oct 22, 2025
@markpollack
Copy link
Member

i can't find reference to this in the spec, can you provide a link?

@JM-Lab
Copy link
Contributor Author

JM-Lab commented Oct 23, 2025

Thank you for pointing this out. I initially thought the official MCP specification mandated a specific path for the SSE message endpoint, but after rechecking, it turns out it does not. According to the official MCP specification, servers should send an endpoint event to inform clients of the URI to use for posting messages.

The motivation for this PR is to align with common practice observed in the MCP ecosystem, where /messages is frequently used for SSE message endpoints:

It improves out-of-the-box interoperability and reduces confusion.

Since SSE is already a deprecated transport in the MCP spec, if keeping /mcp/message is preferred for Spring AI conventions or backward compatibility, I'm happy to close this PR or discuss alternatives.

@ilayaperumalg ilayaperumalg modified the milestones: 1.1.0.M4, 1.1.0.RC1 Oct 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants