Skip to content

[RUM-11854]: Account id and User id propagated in baggage header#2911

Merged
satween merged 1 commit into
developfrom
tvaleev/feature/RUM-11854-account-id_user-id_support
Oct 8, 2025
Merged

[RUM-11854]: Account id and User id propagated in baggage header#2911
satween merged 1 commit into
developfrom
tvaleev/feature/RUM-11854-account-id_user-id_support

Conversation

@satween

@satween satween commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds support for accountId and userId propagation through baggage header.
Add and refactor tests that is related to rum context propagation

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@satween
satween requested review from a team as code owners October 2, 2025 16:30
@codecov-commenter

codecov-commenter commented Oct 2, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.90909% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 70.60%. Comparing base (8612241) to head (44d776f).

Files with missing lines Patch % Lines
.../com/datadog/trace/core/propagation/HttpCodec.java 88.89% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2911      +/-   ##
===========================================
- Coverage    70.64%   70.60%   -0.04%     
===========================================
  Files          815      815              
  Lines        30045    30053       +8     
  Branches      5142     5145       +3     
===========================================
- Hits         21225    21218       -7     
- Misses        7365     7380      +15     
  Partials      1455     1455              
Files with missing lines Coverage Δ
...dog/android/trace/internal/RumContextPropagator.kt 73.21% <100.00%> (-0.86%) ⬇️
.../com/datadog/trace/core/propagation/HttpCodec.java 34.48% <88.89%> (+4.48%) ⬆️

... and 38 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@satween
satween requested a review from ambushwork October 3, 2025 10:55
ambushwork
ambushwork previously approved these changes Oct 3, 2025
@satween
satween force-pushed the tvaleev/feature/RUM-11854-rum-context-fix branch 5 times, most recently from 98981e7 to a8bb138 Compare October 7, 2025 14:07
@satween
satween force-pushed the tvaleev/feature/RUM-11854-account-id_user-id_support branch from 5183a2f to 0cdd0f7 Compare October 7, 2025 15:23
Base automatically changed from tvaleev/feature/RUM-11854-rum-context-fix to develop October 7, 2025 15:24
@satween
satween dismissed ambushwork’s stale review October 7, 2025 15:24

The base branch was changed.

@satween
satween force-pushed the tvaleev/feature/RUM-11854-account-id_user-id_support branch from 0cdd0f7 to 891cb81 Compare October 7, 2025 16:12
@satween
satween force-pushed the tvaleev/feature/RUM-11854-account-id_user-id_support branch from 891cb81 to 44d776f Compare October 7, 2025 16:39
@satween
satween requested a review from ambushwork October 8, 2025 09:34
@satween
satween merged commit a9a1d1e into develop Oct 8, 2025
25 of 26 checks passed
@satween
satween deleted the tvaleev/feature/RUM-11854-account-id_user-id_support branch October 8, 2025 09:36
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.

3 participants