-
Notifications
You must be signed in to change notification settings - Fork 424
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
Network Protection Error Pixels #2019
Conversation
@samsymons I fixed a couple of issues I noticed. See my two commits from today. |
case DispatchSource.MemoryPressureEvent.warning: | ||
Pixel.fire(pixel: .networkProtectionMemoryWarning) | ||
case DispatchSource.MemoryPressureEvent.critical: | ||
Pixel.fire(pixel: .networkProtectionMemoryCritical) |
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.
I'll be curious to see if these ones ever come through - will the extension have enough time to send the pixel before it gets killed? If we start getting warning pixels but never critical ones then that would be my theory, and we can look at implementing an offline pixel cache to catch situations like this.
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.
As far as I can see, this is working really nicely 👍
Co-authored-by: Diego Rey Mendez <[email protected]> Co-authored-by: Dax Mobile <[email protected]> Co-authored-by: Brad Slayter <[email protected]>
Task/Issue URL: https://app.asana.com/0/0/1205365033608088/f
Description:
We want to report errors when the user tries to activate Network Protection on macOS, or when Network Protection encounters an unexpected failure. This includes:
We would like to append a suffix (
_d
|_c
) on each pixel to represent whether or not this was the first time the pixel was fired each day for the user.This is to let us determine whether a spike in pixels is distributed across many devices, or isolated to a small number of devices that are in a bad state and sending pixels repeatedly.
Steps to test this PR:
Testing all the Pixels may be somewhat overkill and, in some cases, not even possible.
m_netp_network_extension_error_activation_request_failed_
which should fire if the user denies VPN permissions)fire(pixelNamed
function—
Internal references:
Software Engineering Expectations
Technical Design Template