Repository navigation
Add HeaderName constants for MCP HTTP headers - #1202
gandhipratik203 wants to merge 1 commit into
Conversation
|
Context for this PR: this came out of downstream integration work in ContextForge, where we needed typed I checked the contribution notes after opening this and saw the "discuss first" preference. Happy to move this into an issue/discussion first if that is preferred. This keeps the existing |
Signed-off-by: Pratik Gandhi <gandhipratik203@gmail.com>
99f9c28 to
38819a8
Compare
|
Quick note: I amended the commit to fix the commitlint failure and pushed the typo-check fix. Because the PR is from a fork, GitHub put CI back into |
|
This has merge conflicts with |
Summary
http::HeaderNameconstants for exact MCP HTTP headers while keeping the existing&strconstants.Mcp-Param-*prefix constant for prefix matching.Validation
cargo test -p rmcp --features client-side-sse,server-side-http --lib transport::common::http_headercargo test -p rmcp --libcargo test -p rmcp --features client,transport-streamable-http-client-reqwest,transport-streamable-http-server --libcargo test -p rmcp --features client,transport-streamable-http-client-unix-socket --libcargo clippy --all-targets --all-features -- -D warningscargo test --all-features