Skip to content

/message break existing usage #501

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Yuan325 opened this issue May 16, 2025 · 0 comments
Open

/message break existing usage #501

Yuan325 opened this issue May 16, 2025 · 0 comments
Labels
bug Something isn't working

Comments

@Yuan325
Copy link

Yuan325 commented May 16, 2025

Describe the bug
A clear and concise description of what the bug is.
This PR (5a25fe3) was recently submitted and break our MCP server's usage with the Inspector. Instead of using the endpoint that was returned (e.g. http://127.0.0.1:5000/mcp?sessionId=123) from /sse endpoint, it updates the endpoint with /messages (http://127.0.0.1:5000/mcp/messages?sessionId=123. This rule was not clearly specified in the mcp docs/specs (only showed in an example).

To Reproduce
Steps to reproduce the behavior:

  1. MCP server with /sse endpoint for GET and / endpoint for POST.

Expected behavior
A clear and concise description of what you expected to happen.
Client should use the endpoint received from GET.

Logs
If applicable, add logs to help explain your problem.
n/a

Additional context
Add any other context about the problem here.

If this is expected, it will be great to include it within the docs/specifications.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant