modelcontextprotocol/java-sdk
Watch
1
0
Fork
You've already forked java-sdk
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: handle empty JSON responses in ResponseSubscribers

Browse source 
- Remove length check in hookOnComplete() to always emit AggregateResponseEvent
- Ensures proper completion handling regardless of response content length
- Add test for empty application/json responses with 200 OK status
This commit is contained in:
codezkk 2025-08-01 16:01:24 +08:00 • committed by Christian Tzolov
commit 5aba8a0e8f
1 changed files with 95 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

7
mcp/src/main/java/io/modelcontextprotocol/client/transport/ResponseSubscribers.java
View file

@ -238,10 +238,9 @@ class ResponseSubscribers {
@Override
protected void hookOnComplete() {
if (this.eventBuilder.length() > 0) {
String data = this.eventBuilder.toString();
this.sink.next(new AggregateResponseEvent(responseInfo, data));
}
String data = this.eventBuilder.toString();
this.sink.next(new AggregateResponseEvent(responseInfo, data));
this.sink.complete();
}