From 75d03f334e15c40b67c5c8867475023af1905fe5 Mon Sep 17 00:00:00 2001 From: Koichi ITO Date: Wed, 18 Mar 2026 12:10:56 +0900 Subject: [PATCH] Use `missing_session_id_response` method in `handle_delete` ## Motivation and Context `handle_delete` had an inline error response for "Missing session ID" that was identical to the existing `missing_session_id_response` helper method. `handle_get` already uses this helper for the same purpose. ## How Has This Been Tested? All existing tests pass. The behavior is unchanged. ## Breaking Change None. --- lib/mcp/server/transports/streamable_http_transport.rb | 8 +------- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/lib/mcp/server/transports/streamable_http_transport.rb b/lib/mcp/server/transports/streamable_http_transport.rb index 6e9dd09..be5404a 100644 --- a/lib/mcp/server/transports/streamable_http_transport.rb +++ b/lib/mcp/server/transports/streamable_http_transport.rb @@ -154,13 +154,7 @@ def handle_delete(request) return success_response end - session_id = request.env["HTTP_MCP_SESSION_ID"] - - return [ - 400, - { "Content-Type" => "application/json" }, - [{ error: "Missing session ID" }.to_json], - ] unless session_id + return missing_session_id_response unless (session_id = request.env["HTTP_MCP_SESSION_ID"]) cleanup_session(session_id) success_response