GeezerVisonThe World Wide Web, restored to 800 pixels of glory. |
| Home|About|Gaming cabinet|Archives|How it works|Starter sites |
GeezerVison is showing github.com — Forward Mcp-Name header in McpTransportContext by neerajbhatt · Pull Request #1326 · modelcontextprotocol/java-sdk Conversation
Contributor
Fixes #1317 MotivationServer features (e.g. routing-header checks for methods like Changes
Contract: the transport only decodes and forwards. For methods it doesn't validate Tests
Focused run: 111 passed, 0 failed.
Sorry, something went wrong. Uh oh!
There was an error while loading. Please reload this page. All reactionsExpose the decoded Mcp-Name request header to server features through a new McpNameHeader value stored under McpNameHeader.TRANSPORT_CONTEXT_KEY. - HTTP: every validated JSON-RPC request carries a value, attached after the user's context extractor so it always wins. The value is either ABSENT, MALFORMED (bad base64, repeated values, or names differing only in case), or the decoded value. - Stdio: carries the NOT_MIRRORED sentinel, so features can tell "no headers on this transport" apart from "client omitted the header". - Decoding reuses McpHeaderDecoder and reads every header name and value through the header accessor. toString() never includes the value. The transport only decodes and forwards. For methods it does not validate itself, the feature owns the equality check against params and should treat a malformed value as a mismatch. Existing validateHeaders() behavior is unchanged. Fixes modelcontextprotocol#1317 Co-Authored-By: Claude Opus 5.5 <[email protected]> This branch has not been deployedNo deployments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
ReviewersNo reviewsUh oh!
There was an error while loading. Please reload this page. AssigneesLabels
None yet
ProjectsMilestoneNo milestoneDevelopmentSuccessfully merging this pull request may close these issues. ForwardMcp-Name header in the McpTransportContext for extension validation
Uh oh!
There was an error while loading. Please reload this page. Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later. |
| You are visitor 01402503 — GeezerVison does not store the pages you look up. Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required. |