Skip to content

Server implementation of Streamable HTTP #461

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
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

RamAnanth
Copy link

@RamAnanth RamAnanth commented Apr 9, 2025

Motivation and Context

This PR implements the Streamable HTTP transport for the MCP protocol (revision 2025-03-26) and addresses the server and tests portion mentioned in #443

How Has This Been Tested?

All existing unit tests are passing. Added a comprehensive unit test to verify different features of the Streamable HTTP Transport requirements.

Breaking Changes

None

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • [] I have added or updated documentation as needed

@RamAnanth RamAnanth closed this Apr 9, 2025
@RamAnanth RamAnanth changed the title Initial version of Streamable HTTP server and testing Server implementation of Streamable HTTP Apr 9, 2025
@RamAnanth RamAnanth reopened this Apr 9, 2025
@zoeshawwang
Copy link

any update? waiting to use.

@RamAnanth
Copy link
Author

I think #554 and #553 would be the PRs to follow for this as they have more functionality

@ferrants
Copy link

If we're looking for a client implementation to test against, this is the PR for the inspector to support Streamable HTTP. It's getting close.
modelcontextprotocol/inspector#339

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