Skip to content

RUM-11136: Move ddtags query parameters to RUM events#2436

Merged
simaoseica-dd merged 2 commits into
hotfix/2.30.1from
simaoseica/RUM-11136/move-session-properties-to-ddtags
Sep 11, 2025
Merged

RUM-11136: Move ddtags query parameters to RUM events#2436
simaoseica-dd merged 2 commits into
hotfix/2.30.1from
simaoseica/RUM-11136/move-session-properties-to-ddtags

Conversation

@simaoseica-dd

@simaoseica-dd simaoseica-dd commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

What and why?

This PR moves the service, sdk_version, version, env, and variant query parameters to RUM events.
This change will ensure that RUM events remain consistently linked to their session values, making persisted batches resilient even if those values change between app launches.

How?

The ddtags from the query parameters are now injected in all RUM events following the format parameter:value separated by commas ,.

Review checklist

  • Feature or bugfix MUST have appropriate tests (unit, integration)
  • Make sure each commit and the PR mention the Issue number or JIRA reference
  • Add CHANGELOG entry for user facing changes
  • Add Objective-C interface for public APIs (see our guidelines (internal) and run make api-surface)

@simaoseica-dd
simaoseica-dd force-pushed the simaoseica/RUM-11136/move-session-properties-to-ddtags branch from 93ee87e to 41233cc Compare August 14, 2025 13:31
@simaoseica-dd
simaoseica-dd changed the base branch from develop to hotfix/2.30.1 September 10, 2025 10:24
@simaoseica-dd
simaoseica-dd force-pushed the simaoseica/RUM-11136/move-session-properties-to-ddtags branch from 41233cc to 6a141f0 Compare September 10, 2025 11:08
@simaoseica-dd
simaoseica-dd marked this pull request as ready for review September 10, 2025 14:23
@simaoseica-dd
simaoseica-dd requested review from a team as code owners September 10, 2025 14:23

@maxep maxep left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@barboraplasovska barboraplasovska left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM👌

@simaoseica-dd
simaoseica-dd merged commit 22a7e37 into hotfix/2.30.1 Sep 11, 2025
16 checks passed
@simaoseica-dd
simaoseica-dd deleted the simaoseica/RUM-11136/move-session-properties-to-ddtags branch September 11, 2025 10:41
@simaoseica-dd simaoseica-dd mentioned this pull request Sep 18, 2025
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