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
⚗️✨ [RUM-2445] implement Tracking Consent management #2589
Conversation
8e5b8c7
to
e73f50b
Compare
Define "start(Rum|Logs)SessionManagerWithDefault" to avoid having to pass new parameters everywhere.
e73f50b
to
2c328ab
Compare
/to-staging |
🚂 Branch Integration: starting soon, merge in < 0s Commit 2c328abf62 will soon be integrated into staging-06. This build is going to start soon! (estimated merge in less than 0s) Use |
🚂 Branch Integration: This commit was successfully integrated Commit 2c328abf62 has been merged into staging-06 in merge commit 8fd5da3faa. Check out the triggered pipeline on Gitlab 🦊 |
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #2589 +/- ##
==========================================
+ Coverage 92.78% 92.79% +0.01%
==========================================
Files 235 236 +1
Lines 6789 6828 +39
Branches 1487 1497 +10
==========================================
+ Hits 6299 6336 +37
- Misses 490 492 +2 ☔ View full report in Codecov by Sentry. |
/to-staging |
🚂 Branch Integration: starting soon, merge in < 8m Commit 2bfb326933 will soon be integrated into staging-06. This build is going to start soon! (estimated merge in less than 8m) Use |
🚂 Branch Integration: This commit was successfully integrated Commit 2bfb326933 has been merged into staging-06 in merge commit d01c92fed2. Check out the triggered pipeline on Gitlab 🦊 |
Motivation
Allows starting/stopping data collection based on user consent (ex: regarding to RGPD rules and such).
Related issue: #1008
Changes
For both RUM and Logs SDKs:
trackingConsent: "granted"/"not-granted"
initialization parameter (defaults to"granted"
for backward compatibility)setTrackingConsent
method to set the consent state before or after callinginit()
. If called beforeinit()
, the initialization parameter will be ignored.Testing
I have gone over the contributing documentation.