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
Add additional telemetry for WAM bridge #5308
Conversation
…tion-library-for-js into add-static-fields-telemetry
Codecov Report
*This pull request uses carry forward flags. Click here to find out more.
|
matsAccountJoinOnStart: mats ? mats.account_join_on_start : undefined, | ||
matsAccountJoinOnEnd: mats ? mats.account_join_on_end : undefined, | ||
matsDeviceJoin: mats ? mats.device_join : undefined, | ||
matsPromptBehavior: mats ? mats.prompt_behavior : undefined, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is all MATS data considered static?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, we only get MATS data back once per request
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved to merge. One q: Are we considering everything that we get some MATS static?
🎉 Handy links: |
🎉 We recommend upgrading to the latest version of Handy links: |
* Add static fields to telemetry * resolve merge conflicts * restore removed params * Fix tests * Change files * Update RefreshTokenClient.spec.ts * add mats fields to telemetry * fix test * restore API surface for performance callback * Change files
* Add static fields to telemetry * resolve merge conflicts * restore removed params * Fix tests * Change files * Update RefreshTokenClient.spec.ts * add mats fields to telemetry * fix test * restore API surface for performance callback * Change files
Adds additional fields to our telemetry when going through the native broker.
Replaces #5098
This follows #5224