Standard header name used in mcp#320
Standard header name used in mcp#320CrazyHZM wants to merge 4 commits intomodelcontextprotocol:mainfrom
Conversation
Signed-off-by: JermaineHua <crazyhzm@apache.org>
# Conflicts: # mcp-spring/mcp-spring-webflux/src/main/java/io/modelcontextprotocol/client/transport/WebClientStreamableHttpTransport.java
| /** | ||
| * Standard header name used in MCP JSON-RPC request and responses. | ||
| */ | ||
| public static final class Headers { |
There was a problem hiding this comment.
I feel that this belongs around something like McpHttpTransport interface rather than in the McpSpec, which is more about the JSON-RPC layer.
There was a problem hiding this comment.
The primary purpose here is to provide users with the MCP standard header name information. The MCP-Session-Id does belong to the transport category. Of course, I have omitted the last-event-id here. At present, I think defining these header names as part of the MCPSchema is the most reasonable approach because I haven't thought of any other more reasonable alternatives. These header names are indeed mentioned in the Specification. Do you have any better suggestions? @chemicL
chemicL
left a comment
There was a problem hiding this comment.
Thanks, please review the comments.
# Conflicts: # mcp-spring/mcp-spring-webflux/src/main/java/io/modelcontextprotocol/client/transport/WebClientStreamableHttpTransport.java
|
Thanks you @CrazyHZM , |
Motivation and Context
How Has This Been Tested?
Breaking Changes
Types of changes
Checklist
Additional context