-
Notifications
You must be signed in to change notification settings - Fork 274
feat(client): adds StreamableHttpClientTransport #144
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
base: main
Are you sure you want to change the base?
feat(client): adds StreamableHttpClientTransport #144
Conversation
mcp/src/main/java/io/modelcontextprotocol/client/transport/StreamableHttpClientTransport.java
Outdated
Show resolved
Hide resolved
bb851b3
to
8b593e0
Compare
mcp/src/main/java/io/modelcontextprotocol/client/transport/StreamableHttpClientTransport.java
Outdated
Show resolved
Hide resolved
Great job @Aliaksie Here are some of my thoughts after reading this PR and the SDK source code. I think just adding the McpClientTransport implementation cannot give full support for Streamable HTTP because how it works is inconsistent with HTTP+SSE in many places, which might require some redesign or refactoring of the current SDK implementation:
|
f07a466
to
67b2881
Compare
Hey @chickenlj Thanks a lot for the thoughtful feedback! 1 I added to Streamable HTTP transport code to manage I hope all this makes the SDK compliant with Streamable HTTP's flexibility while still maintaining support for HTTP+SSE fallback |
…y StreamableHttpClientTransport
a677cb1
to
c1f0a93
Compare
adds StreamableHttpClientTransport
Motivation and Context
it is enabler for support 2025-03-26 version of protocol
How Has This Been Tested?
Breaking Changes
No
Types of changes
Checklist
Additional context
The server implementation and the 2025-03-26 scheme will be added as separate PRs.