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

SDK Parity: Avoid Parsing Server Response for non-JsonRPCMessage Requests (#1290)

Browse source
This commit is contained in:
Justin Wang 2025-08-26 09:14:47 -06:00 • committed by GitHub
commit 9c6fd15a88
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
1 changed files with 163 additions and 11 deletions
Download patch file Download diff file Expand all files Collapse all files

24
src/mcp/client/streamable_http.py
View file

@ -279,17 +279,19 @@ class StreamableHTTPTransport:
if is_initialization:
self._maybe_extract_session_id_from_response(response)
content_type = response.headers.get(CONTENT_TYPE, "").lower()
if content_type.startswith(JSON):
await self._handle_json_response(response, ctx.read_stream_writer, is_initialization)
elif content_type.startswith(SSE):
await self._handle_sse_response(response, ctx, is_initialization)
else:
await self._handle_unexpected_content_type(
content_type,
ctx.read_stream_writer,
)
# Per https://modelcontextprotocol.io/specification/2025-06-18/basic#notifications:
# The server MUST NOT send a response to notifications.
if isinstance(message.root, JSONRPCRequest):
content_type = response.headers.get(CONTENT_TYPE, "").lower()
if content_type.startswith(JSON):
await self._handle_json_response(response, ctx.read_stream_writer, is_initialization)
elif content_type.startswith(SSE):
await self._handle_sse_response(response, ctx, is_initialization)
else:
await self._handle_unexpected_content_type(
content_type,
ctx.read_stream_writer,
)
async def _handle_json_response(
self,