Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Comments

Close side panel

JDK HTTP Client Streamable HTTP support#317

Closed
taobaorun wants to merge 3 commits intomodelcontextprotocol:mainmodelcontextprotocol/java-sdk:mainfrom
taobaorun:feature/jdkhttpclient_streamabletaobaorun/java-sdk:feature/jdkhttpclient_streamableCopy head branch name to clipboard
Closed

JDK HTTP Client Streamable HTTP support#317
taobaorun wants to merge 3 commits intomodelcontextprotocol:mainmodelcontextprotocol/java-sdk:mainfrom
taobaorun:feature/jdkhttpclient_streamabletaobaorun/java-sdk:feature/jdkhttpclient_streamableCopy head branch name to clipboard

Conversation

@taobaorun
Copy link

@taobaorun taobaorun commented Jun 16, 2025 •
edited
Loading

Motivation and Context

JDK HTTP Client-based client transport for Streamable HTTP. Reference the WebClient Streamable HTTP implementation. #292

How Has This Been Tested?

  • Regular integration tests using the everything MCP server.

Breaking Changes

The client initialization became lazy and more error-prone - connections happen upon first request and initialization is performed implicitly if it has not been issued before.

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

Additional context

Related #310

@noear
Copy link

noear commented Jun 18, 2025

good job

@tzolov tzolov added the duplicate This issue or pull request already exists label Jul 1, 2025
@tzolov
Copy link
Contributor

tzolov commented Jul 1, 2025

Duplicate of #337

@tzolov tzolov marked this as a duplicate of #337 Jul 1, 2025
@tzolov
Copy link
Contributor

tzolov commented Jul 1, 2025

@taobaorun Thank you for your contribution and effort on this feature!
We're currently consolidating our work on the HTTP client-based streamable http client transport in PR #337 to focus our limited resources effectively.
Would you mind taking a look at that PR and sharing any feedback or concerns you might have? Your input would be valuable.
Thanks again for taking the initiative on this!

@taobaorun
Copy link
Author

taobaorun commented Jul 1, 2025

@taobaorun Thank you for your contribution and effort on this feature! We're currently consolidating our work on the HTTP client-based streamable http client transport in PR #337 to focus our limited resources effectively. Would you mind taking a look at that PR and sharing any feedback or concerns you might have? Your input would be valuable. Thanks again for taking the initiative on this!

I've been waiting for this feature. Thank you. 🤝

@taobaorun taobaorun closed this Jul 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

duplicate This issue or pull request already exists

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@taobaorun @noear @tzolov
Morty Proxy This is a proxified and sanitized view of the page, visit original site.