Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(tracing) Collect DSC for head of trace #457

Closed
16 tasks
smeubank opened this issue Jun 28, 2022 · 0 comments · Fixed by #492
Closed
16 tasks

feat(tracing) Collect DSC for head of trace #457

smeubank opened this issue Jun 28, 2022 · 0 comments · Fixed by #492

Comments

@smeubank
Copy link
Member

Describe the idea

Applications instrumented with a Sentry SDK should collect and propagate dynamic sampling context data to all outgoing requests and to Sentry.

Requirement:

When an application has an event which is the head of a trace,
the SDK shall collect and propagate all possible values in outgoing requests and to Sentry
so that contextual data can be propagated in all connected services across the continuation of trace

what should be sent?

baggage header

  • sentry-trace_id
  • sentry-public_key
  • sentry-sample_rate
  • sentry-release
  • sentry-environment
  • sentry-user_id
  • sentry-user_segment
  • sentry-transaction

Trace Envelope Header

  • trace_id (string)
  • public_key (string)
  • sample_rate (string)
  • release (string)
  • environment (string)
  • user_id (string)
  • user_segment (string)
  • transaction (string)

where should it be sent?

  • in dynamic sample context (by baggage)
  • in the envelope header
  • meta data (was/is being done in JS to be check if still necessary)

Example:
Here is an example from the Sentry UI, where the implementation has already begun on the latest versions of the JS SDK. A request from the front end to a down stream applications includes in the request header baggage

image

Why do you think it's beneficial to most of the users

This ensures that further downstream applications connected in a distributed trace will receive all contextual data required for dynamically sampling.

Possible implementation

Implementation has already begun (at the time of this creation) on JS and Android. Refer to the SDK Developer documentation for further information

For full up to date requirements according to SDK Developer specifications please see the associated documentation.

Dynamic Sampling Context

Original PR for full context
Add spec for Dynamic Sampling Context #613

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants