-
Notifications
You must be signed in to change notification settings - Fork 113
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
VPN-2988 - Record events on controller error steps #4594
Conversation
DATA REVIEW REQUEST
How often are users experiencing server unavailable errors.
This is important for the VPN team to monitor these errors more closely.
None.
No.
This collection is Glean so is documented in the Glean Dictionary.
This collection will be collected permanently.
All channels, countries, and locales. No filters.
These collections are Glean. The opt-out can be found in the product's preferences.
The Mozilla VPN team will regularly monitor this data thorugh Looker dashboards.
Within the Mozilla VPN team and Security and Privacy Products team.
No. |
Codecov ReportBase: 70.49% // Head: 29.19% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #4594 +/- ##
===========================================
- Coverage 70.49% 29.19% -41.30%
===========================================
Files 255 248 -7
Lines 16009 13833 -2176
Branches 8104 7915 -189
===========================================
- Hits 11285 4039 -7246
+ Misses 4323 3956 -367
- Partials 401 5838 +5437
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
c95bfe1
to
e0782d6
Compare
DATA COLLECTION REVIEW RESPONSE:
Yes.
Yes. This collection is Telemetry so can be controlled through Firefox's Preferences.
Yes, Bea is responsible.
Category 1, Technical.
Default on for all channels.
No.
Yes.
No. Result: datareview+ |
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.
Apply this change because I suspect some platforms/compilers can complain.
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.
No description provided.