Skip to content

fix: unsupported header takes precedence over header mismatch - #293

Merged
pcarleton merged 1 commit into
modelcontextprotocol:mainfrom
Yuan325:fix-sep-2575
May 20, 2026
Merged

fix: unsupported header takes precedence over header mismatch#293
pcarleton merged 1 commit into
modelcontextprotocol:mainfrom
Yuan325:fix-sep-2575

Conversation

@Yuan325

Copy link
Copy Markdown
Contributor

The sep-2575-http-server-header-mismatch-400 should check with a supported header and mismatch protocol version in _meta.

According to the SEP,

* Requirement: This header is MANDATORY. Servers should reject requests with a missing or unsupported version.
* This header MUST match the value provided in the Request as specified below.

hence, unsupported header takes precedence over header mismatch. In order to run this check, we must provide a supported header version with a mismatch _meta.protocolVersion value.

@pkg-pr-new

Copy link
Copy Markdown

Open in StackBlitz

npx https://pkg.pr.new/@modelcontextprotocol/conformance@293

commit: 77594b4

@Yuan325

Copy link
Copy Markdown
ContributorAuthor

@pcarleton If you can take a look at this that would be great~ Thank you!

@pcarletonpcarleton left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lgtm, ty!

@pcarleton
pcarleton merged commit 0371b18 into modelcontextprotocol:mainMay 20, 2026
4 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@Yuan325@pcarleton