-
Notifications
You must be signed in to change notification settings - Fork 44
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
feat: Add rum-response metadata handler #913
Conversation
Asset Size Report
Merging this pull request will result in the following asset size changes:
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #913 +/- ##
==========================================
- Coverage 80.42% 80.40% -0.02%
==========================================
Files 154 154
Lines 7044 7048 +4
Branches 1386 1388 +2
==========================================
+ Hits 5665 5667 +2
- Misses 1163 1165 +2
Partials 216 216
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
The initial connect call can now supply browser app metadata. This adds a handler to store and use that metadata for future harvests.
Overview
This PR introduces a handler for the agent metadata response from the RUM call. This will initially contain a real entityGuid for the browser app, regardless if it was APM injected or standalone. The entityGuid implementations will be introduced in subsequent PRs, as it will be applied to SR, ST and Logs payloads in the future.
Related Issue(s)
NR-215487
Testing
A new test has been added to account for the rum-supplied app id.