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

✨[RUMF-465] collect client service, env and version #392

Merged
merged 7 commits into from
May 18, 2020

Conversation

mquentin
Copy link
Contributor

Motivation

Allows users to configure optional fields that tag their RUM events and logs with a specific service, environment and version.

Changes

New optional configuration options have been added to the log, rum init process. If define, these new fields will then create new tags for each of the logs and RUM events send to the intake.

Testing

  1. add the optional clientService, clientEnv, clientVersion fields to the window.DD_RUM.init and window.DD_LOGS.init
  2. check in the network tab that the intake URL carries the new tags ex: &ddtags=sdk_version:dev,env:envaaa,service:serviceaaa,version:versionaaa

@mquentin mquentin requested a review from a team as a code owner May 15, 2020 09:41
packages/core/src/configuration.ts Outdated Show resolved Hide resolved
packages/core/src/configuration.ts Outdated Show resolved Hide resolved
packages/core/src/configuration.ts Show resolved Hide resolved
packages/core/src/configuration.ts Outdated Show resolved Hide resolved
Copy link
Contributor

@bcaudan bcaudan left a comment

Choose a reason for hiding this comment

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

this new API should be tested

@mquentin mquentin changed the title ✨ - SDK collect client service, env and version ✨ - [RUMF-465] collect client service, env and version May 15, 2020
@bcaudan bcaudan changed the title ✨ - [RUMF-465] collect client service, env and version ✨[RUMF-465] collect client service, env and version May 18, 2020
@mquentin mquentin merged commit 723100b into master May 18, 2020
@mquentin mquentin deleted the maxime.quentin/SDKcollectServiceEnvVersion branch May 18, 2020 11:26
@bcaudan bcaudan mentioned this pull request Aug 24, 2020
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.

2 participants