Skip to content

mcp: accept parameterized Accept media types#853

Merged
maciej-kisiel merged 2 commits intomodelcontextprotocol:mainfrom
kalvinnchau:main
Mar 31, 2026
Merged

mcp: accept parameterized Accept media types#853
maciej-kisiel merged 2 commits intomodelcontextprotocol:mainfrom
kalvinnchau:main

Conversation

@kalvinnchau
Copy link
Copy Markdown
Contributor

@kalvinnchau kalvinnchau commented Mar 23, 2026

Normalize accept tokens before validation so valid headers with media type parameters still match their base media types.

This allows the streamable mcp server to validate the following header Accept: application/json;charset=utf-8, text/event-stream

kalvinnchau and others added 2 commits March 23, 2026 15:54
Normalize accept tokens before validation so valid headers with media type
parameters still match their base media types.

Add test coverage for parameterized and split accept headers.
Copy link
Copy Markdown
Contributor

@maciej-kisiel maciej-kisiel left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@maciej-kisiel maciej-kisiel merged commit c1a12cf into modelcontextprotocol:main Mar 31, 2026
9 checks passed
Sign up for free to 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