Skip to content

OpenTelemetry extract trace context from _meta in incoming requests - #2133

Closed
aabmass wants to merge 5 commits into
modelcontextprotocol:mainfrom
aabmass:otel-propagation-extract
Closed

OpenTelemetry extract trace context from _meta in incoming requests#2133
aabmass wants to merge 5 commits into
modelcontextprotocol:mainfrom
aabmass:otel-propagation-extract

Conversation

@aabmass

Copy link
Copy Markdown
Contributor

Part of #421

Depends on

Motivation and Context

Extract's OTel context from _meta. This is what lets span parent/child relationship happen.

How Has This Been Tested?

New test cases

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

Additional context

@aabmassaabmass changed the title Otel propagation extractOpenTelemetry extract trace context from _meta in incoming requestsFeb 24, 2026
@aabmass
aabmassforce-pushed the otel-propagation-extract branch 4 times, most recently from 23b5cd5 to 0ebbf21CompareMarch 3, 2026 06:14
See if tests still fail
This fix covers in memory, streamable http, and sse transports.
This works for both incoming client->server requests and server->client requests like MCP Sampling.
@aabmass
aabmassforce-pushed the otel-propagation-extract branch from 0ebbf21 to c0ae32fCompareMarch 6, 2026 05:29
@Kludex

Copy link
Copy Markdown
Member

We already have this.

@KludexKludex closed this Jun 25, 2026
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

@aabmass@Kludex