health checkpt added #121
health checkpt added #121
Conversation
There was a problem hiding this comment.
We might not require fastapi, we can just leverage existing packages. Ex. https://gofastmcp.com/deployment/running-server#custom-routes
| ) | ||
| parser.add_argument( | ||
| "--path", type=str, default="/", help="Path of the streamable HTTP server" | ||
| "--path", type=str, default="/mcp/", help="Path of the streamable HTTP server" |
There was a problem hiding this comment.
Also any specific reason for this change?
There was a problem hiding this comment.
The MCP server’s endpoint is mounted at /mcp/ for Streamable HTTP transport and /sse/ for SSE transport if there is no argument
Link
, so this helps standardise things
also /health does'nt work if mcp server is also running at root
There was a problem hiding this comment.
How does it standardise things? /health should work no matter what the base path is
There was a problem hiding this comment.
The default for fastmcp if we dont pass the default argument as / is /mcp/
and /health doesnt work if mcp is run at root because if fastmcp server is started at / as
base path then any end point after / will go to fastmcp standard
this happens when theyre at the same path
When FastMCP runs at the root path (/), all HTTP requests go through the MCP protocol handler first, which requires the mcp-session-id header.
Added Health Checkpoint as requested in Discussion
changed default path from
/to/mcp/