FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[WIP] Refactor HttpClient use to avoid leaking references by Kehrlann · Pull Request #1079 · modelcontextprotocol/java-sdk · GitHub

[WIP] Refactor HttpClient use to avoid leaking references - #1079

Draft
Kehrlann wants to merge 1 commit into
mainfrom
dgarnier/issue-620-refactor-http-client-chains
Draft

[WIP] Refactor HttpClient use to avoid leaking references#1079
Kehrlann wants to merge 1 commit into
mainfrom
dgarnier/issue-620-refactor-http-client-chains

Conversation

Kehrlann commented Aug 7, 2026
edited
Loading

Copy link
Copy Markdown
Contributor

Refactor HttpClient use to avoid capturing the sseSink that leads to leaking the httpClient reference due to a cycle.

Rebasing @chemicL 's work on issue #547 & #620 . This should also solve performance issues in #1042

…leaking the httpClient reference due to a cycle

Signed-off-by: Daniel Garnier-Moiroux <git@garnier.wf>
Signed-off-by: Dariusz Jędrzejczyk <2554306+chemicL@users.noreply.github.com>
Kehrlann added this to the 2.1.0 Planning milestone Aug 7, 2026
Kehrlann self-assigned this Aug 7, 2026
Kehrlann added area/client P1 Significant bug affecting many users, highly requested feature area/transport do not merge labels Aug 7, 2026
Kehrlann force-pushed the dgarnier/issue-620-refactor-http-client-chains branch from 16fee90 to 30dc4d3 Compare August 7, 2026 16:14
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/client area/transport do not merge P1 Significant bug affecting many users, highly requested feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL