Skip to content

Conversation

@swcollard
Copy link
Contributor

Inside the reqwest middleware, if the global text_map_propagator is not set, it will no op and not send the traceparent and tracestate headers to the Router. Adding this is needed to correlate traces from the mcp server to the router or other downstream APIs

@swcollard swcollard requested a review from a team as a code owner August 28, 2025 17:40
@apollo-librarian
Copy link

apollo-librarian bot commented Aug 28, 2025

✅ Docs preview ready

The preview is ready to be viewed. View the preview

File Changes

0 new, 7 changed, 0 removed
* (developer-tools)/apollo-mcp-server/(latest)/command-reference.mdx
* (developer-tools)/apollo-mcp-server/(latest)/index.mdx
* (developer-tools)/apollo-mcp-server/(latest)/install.mdx
* (developer-tools)/apollo-mcp-server/(latest)/limitations.mdx
* (developer-tools)/apollo-mcp-server/(latest)/quickstart.mdx
* (developer-tools)/apollo-mcp-server/(latest)/guides/auth.mdx
* (developer-tools)/apollo-mcp-server/(latest)/_sidebar.yaml

Build ID: aab62dd1483c101300c043d6
Build Logs: View logs

URL: https://www.apollographql.com/docs/deploy-preview/aab62dd1483c101300c043d6

@github-actions
Copy link

Changeset file missing for PR

All changes should include an associated changeset file.
Please refer to README for more information on generating changesets.

@swcollard swcollard requested review from DaleSeo and alocay August 28, 2025 17:56
Copy link
Contributor

@DaleSeo DaleSeo left a comment

Choose a reason for hiding this comment

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

LGTM!

@swcollard swcollard merged commit d946b25 into feature/telemetry Aug 28, 2025
8 checks passed
@swcollard swcollard deleted the telemetry/fix-downstream-traces branch August 28, 2025 19:05
swcollard added a commit that referenced this pull request Sep 2, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
swcollard added a commit that referenced this pull request Sep 5, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
swcollard added a commit that referenced this pull request Sep 17, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
swcollard added a commit that referenced this pull request Sep 23, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
swcollard added a commit that referenced this pull request Sep 24, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
@apollo-bot2 apollo-bot2 mentioned this pull request Sep 24, 2025
@DaleSeo DaleSeo mentioned this pull request Sep 24, 2025
DaleSeo pushed a commit that referenced this pull request Sep 24, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
DaleSeo pushed a commit that referenced this pull request Sep 24, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
DaleSeo pushed a commit that referenced this pull request Sep 29, 2025
…l headers downstream (#307)

* Fix sending OTLP trace headers downstream to GQL API

* Changeset
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.

4 participants