Skip to content

Streamable HTTP also needs a GET/DELETE request for notifications and messages from the server #99

@paoloricciuti

Description

@paoloricciuti

In the code for the mcp handler there's these lines

https://github.com/vercel/mcp-adapter/blob/0b92b2e915108e24cefb9df5667f6befff840737/src/handler/mcp-api-handler.ts#L270-L298

However this is wrong: the GET method is used by the mcp client to instantiate an SSE connection so that the server can send notifications and messages (like elicitation/sampling/roots) to the client.

DELETE is even worse because that's the request sent when the client disconnects from the server.

I suppose the stremamable http would need the same kind of treatment as the SSE transport with REDIS acting as a pub/sub to send the notifications and messages back to the client.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions