Skip to content

Capture sentry-trace with the correct http header key#1260

Merged
st0012 merged 4 commits intomasterfrom
fix-sentry-trace-header
Feb 2, 2021
Merged

Capture sentry-trace with the correct http header key#1260
st0012 merged 4 commits intomasterfrom
fix-sentry-trace-header

Conversation

@st0012
Copy link
Copy Markdown
Contributor

@st0012 st0012 commented Feb 1, 2021

When receiving a request with `sentry-trace` header, the server will alter
the header's key to `HTTP_SENTRY_TRACE`. So we should use
`HTTP_SENTRY_TRACE` to capture the trace data in the middleware.

When receiving a request with `sentry-trace` header, the server will alter
the header's key to `HTTP_SENTRY_TRACE`. So we should use
`HTTP_SENTRY_TRACE` to capture the trace data in the middleware.
@st0012 st0012 force-pushed the fix-sentry-trace-header branch from 8a7aef9 to 6befa80 Compare February 1, 2021 07:06
@st0012 st0012 merged commit fb2679a into master Feb 2, 2021
@st0012 st0012 deleted the fix-sentry-trace-header branch February 2, 2021 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant