Skip to content

Commit 770ce20

Browse files
author
Alberto Farah
committed
fix: add warning log when session ID is unknown or expired
Closes #2204 The else branch in _handle_stateful_request() silently returned 404 when receiving an unknown or expired session ID. The other branches already log at debug/info level. This adds a warning log to help operators diagnose stale session ID issues (e.g. after server restarts).
1 parent b33c811 commit 770ce20

1 file changed

Lines changed: 4 additions & 0 deletions

File tree

src/mcp/server/streamable_http_manager.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -270,6 +270,10 @@ async def run_server(*, task_status: TaskStatus[None] = anyio.TASK_STATUS_IGNORE
270270
await http_transport.handle_request(scope, receive, send)
271271
else:
272272
# Unknown or expired session ID - return 404 per MCP spec
273+
logger.warning(
274+
"Rejected request with unknown or expired session ID: %s",
275+
session_id,
276+
)
273277
# TODO: Align error code once spec clarifies
274278
# See: https://github.com/modelcontextprotocol/python-sdk/issues/1821
275279
error_response = JSONRPCError(

0 commit comments

Comments
 (0)