getsentry / getsentry/sentry-rust

[Dynamic Sampling] Basic support

Open
#741 0 comments 0 reactions 0 assignees View on GitHub
Feature Rust SDK parity Traces
Dominant language
Rust
Stars
752
Forks
190
Avg merge
1h 21m
Merged PRs (30d)
5

Description

## Description

We want the SDK to support Dynamic Sampling.
For this to happen, we need to:

* create the necessary data structures (DSC)
* DSC added in [https://github.com/getsentry/sentry-rust/pull/867/files]() and `TransactionMetadata` in a follow-up PR to that one. Maybe we'll need more.
* collect the DSC from incoming requests
* Propagating DSC to downstream SDKs (via the `baggage` header).
* send out the DSC when sending the envelopes to Sentry
* we needed to send out part of it to solve an issue with metrics extrapolation, as the backend would assume an incorrect sample rate when it was not sent out from the SDK [https://github.com/getsentry/sentry-rust/pull/869]()

As we are not instrumenting any outgoing network requests, we are not concerned with serializing the DSC into the baggage header or propagating it to other SDKs.

[Spec]()

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.