-
Notifications
You must be signed in to change notification settings - Fork 4
Add telemetry events #152
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
Draft
jonathanmendez
wants to merge
35
commits into
enterprise-main
Choose a base branch
from
add-more-telemetry-events
base: enterprise-main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Add telemetry events #152
Changes from 1 commit
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
3b4a633
WIP
gcp e38d646
feat: add enterprise download security telemetry with MOZ_ENTERPRISE …
gcp ea2854a
feat: add enterprise download telemetry with policy controls
gcp e7f5908
chore: update Cargo.lock
gcp 02f5632
feat: add TELEMETRY_ENDPOINT env var support for telemetry endpoints
gcp 61dd192
refactor: unify TELEMETRY_ENDPOINT handling across components
gcp 3205509
refactor: use lazy loading for DownloadsTelemetry
gcp 4caf74b
feat: add localhost support for telemetry testing and debugging
gcp 274abf1
feat: prioritize TELEMETRY_ENDPOINT env var over localhost config
gcp 150fcc7
feat: add debug logging for download telemetry
gcp 038925f
fix: Improve error handling and resource path resolution in downloads…
gcp 47cea1a
feat: wrap file_downloaded telemetry in MOZ_ENTERPRISE ifdef
gcp e484c38
fix: use MOZ_TELEMETRY_REPORTING instead of MOZILLA_OFFICIAL for tele…
gcp b1f9540
feat: reset telemetry localhost port and expose MOZ_TELEMETRY_REPORTI…
gcp cc580c9
refactor: move download telemetry to dedicated enterprise ping
gcp ca80612
fix: use PathUtils as global instead of importing as ES module
gcp 6997c65
POC metric for page printed
19b231e
Update metric to have better data
72a29ea
Put print telemetry behind #ifdef MOZ_ENTERPRISE
f824376
Add and check prefs for metric
384b4a4
Add content titles to print metric payload
e6a22fb
Add existing add-on install metric to enterprise pings
f98868d
Include info section in enterprise ping so console can consume
da7695d
Allow enterprise prefix policies to be set via policy
0bd4adb
Enterprise glean client
fiji-flo 6ee43ad
Add metric for browsing a blocked URL
f3bf7a2
Remove debugging force-enable of download telemetry
65185b4
Add policy and pref for telemetry of browsing blocked domains
2090a56
Protect against telemetry exceptions for printing
eaaa96a
Protect against telemetry exceptions during website filtering
9b74d2f
Add BlocklistDomainBrowsedTelemetry policy to schema
9e4617a
Revert defunct changes to redirect telemetry
d72789d
Remove defunct cargo dependency on local glean
7da0388
Remove obsolete pref
4b6aecb
Remove trace log statements
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@jonathanmendez This one is confusing me, I don't think this patch adds any preferences under that prefix?