-
Notifications
You must be signed in to change notification settings - Fork 62
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
Privacy Config: Update "featuresRequestHeader" configuration for versions 5.222.0 and 5.223.0 #2602
Conversation
…ations for versions 52220000 and 52230000.
Don't forget to assign an individual reviewer (in addition to those already added, this should create a task for them in Asana). |
Generated file outputs:Time updated: Thu, 19 Dec 2024 20:01:29 GMT legacytrackers-unprotected-temporary.txt (13 more)
v3/android-config.json--- v3/android-config.json
+++ v3/android-config.json
@@ -199,11 +199,33 @@
"daysLogging": 10,
"featuresLogged": {
"gpc": true,
"cpm": true,
- "appTp": false,
+ "appTP": false,
"netP": false
}
+ },
+ {
+ "appVersion": 52220000,
+ "daysAfterBuild": 6,
+ "daysLogging": 28,
+ "featuresLogged": {
+ "gpc": false,
+ "cpm": false,
+ "appTP": true,
+ "netP": false
+ }
+ },
+ {
+ "appVersion": 52230000,
+ "daysAfterBuild": 6,
+ "daysLogging": 10,
+ "featuresLogged": {
+ "gpc": false,
+ "cpm": true,
+ "appTP": false,
+ "netP": false
+ }
}
]
}
}
latestv4/android-config.json--- v4/android-config.json
+++ v4/android-config.json
@@ -190,11 +190,33 @@
"daysLogging": 10,
"featuresLogged": {
"gpc": true,
"cpm": true,
- "appTp": false,
+ "appTP": false,
"netP": false
}
+ },
+ {
+ "appVersion": 52220000,
+ "daysAfterBuild": 6,
+ "daysLogging": 28,
+ "featuresLogged": {
+ "gpc": false,
+ "cpm": false,
+ "appTP": true,
+ "netP": false
+ }
+ },
+ {
+ "appVersion": 52230000,
+ "daysAfterBuild": 6,
+ "daysLogging": 10,
+ "featuresLogged": {
+ "gpc": false,
+ "cpm": true,
+ "appTP": false,
+ "netP": false
+ }
}
]
}
}
v4/extension-brave-config.json (12 more)
|
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.
looks good, just one change on the logging days for 5.222.0
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.
LGTM
Asana Task/Github Issue: https://app.asana.com/0/0/1209015004618387/f
Description
Updated featuresRequestHeader. Renamed appTp to appTP. Added configurations for versions 52220000 and 52230000.
Feature change process:
Additional info:
Site breakage mitigation process:
Brief explanation
Reported URL:
Problems experienced:
Platforms affected:
Tracker(s) being unblocked:
Feature being disabled:
I have referenced the URL of this PR as the "reason" value for the exception (where applicable).
This change is a speculative mitigation to fix reported breakage.
Reference