Skip to content

Fix protocol version tag for stateless server spans - #1810

Open
luisangelrod wants to merge 3 commits into
modelcontextprotocol:mainfrom
luisangelrod:fix/protocol-version-activity-tag
Open

Fix protocol version tag for stateless server spans#1810
luisangelrod wants to merge 3 commits into
modelcontextprotocol:mainfrom
luisangelrod:fix/protocol-version-activity-tag

Conversation

@luisangelrod

Copy link
Copy Markdown
Contributor

Summary

  • Use the per-request protocol version when tagging server activities before session-level negotiation state is populated
  • Add stateless HTTP coverage for the 2026-07-28 protocol version tag

Fixes #1800

Testing

  • Focused stateless protocol-version activity test: 1 passed
  • ModelContextProtocol.Tests DiagnosticTests on net10.0: 3 passed
  • MapMcpStatelessTests on net10.0: 32 passed, 20 skipped as expected

@jeffhandley

Copy link
Copy Markdown
Contributor

Thanks for submitting this, @luisangelrod! Tagging @tarekgh for review.

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.

mcp.protocol.version activity tag not emitted on server spans in stateless HTTP mode

2 participants