Revert "Add message queue for SSE messages POST endpoint (#459)" #649
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
ihrpr
commented
May 7, 2025
ihrpr
commented
- In spec version 2025-03-26, Streamable HTTP transport supersedes the HTTP+SSE transport from version 2024-11-05
- We're bringing the Python SDK to the new spec version (2025-03-26), which means focusing on Streamable HTTP rather than enhancing the legacy transport, unless it's a fix
- Adding Redis integration to SSE creates maintenance burden for a transport being superseded. It would be still useful to have an MCP Server with Redis but as an example so users can reference it.
bhosmer-ant
requested review from
Kludex and
bhosmer-ant
and removed request for
Kludex
bhosmer-ant
approved these changes
May 7, 2025
|
👍 I really didn't understand why merging the redis stuff. |
I saw that SSE was deprecated but had still put some time into it so wanted to get it merged 😅 . It seems like this potentially could be useful for stateful streamable HTTP still, so maybe it can get revived eventually. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.