-
Notifications
You must be signed in to change notification settings - Fork 1.3k
For #11704 Add leanplum attributes for tracking protection #11965
Conversation
Request for data collection review formAll questions are mandatory. You must receive review from a data steward peer on your responses to these questions before shipping new data collection.
To measure how often tracking protection is enabled/disabled.
To measure how often tracking protection is enabled/disabled, to help direct users discover ETP controls; this is in the context of a CFR like message experiment.
This data will be present as long as the user ID exists in Leanplum.
|
@jhugman since there isn't an expiry for LeanPlum in the SDK, can you list who would be monitoring this data, and would be the contact for expiring/renewing if need be? I'll need that to complete the data review form. Also wondering if it LP would do expiry for us at some point... @cbonacuse or could you answer this question ^? |
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.
Looking good! Just a few questions :)
app/src/main/java/org/mozilla/fenix/components/metrics/LeanplumMetricsService.kt
Outdated
Show resolved
Hide resolved
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.
Data review+ only
Data Review Form (to be filled by Data Stewards)
- Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?
Yes, this is documented in docs/mma.md
- Is there a control mechanism that allows the user to turn the data collection on and off?
Yes, Leanplum is controlled by the data controls in Fenix settings
- If the request is for permanent data collection, is there someone who will monitor the data over time?
@cbonacuse will monitor this, and we will also look into Leanplum expiry, or check in every 6mo
- Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
Type 2, user interaction with tracking protection setting
- Is the data collection request for default-on or default-off?
default on
- Does the instrumentation include the addition of any new identifiers (whether anonymous or otherwise; e.g., username, random IDs, etc. See the appendix for more details)?
No
- Is the data collection covered by the existing Firefox privacy notice?
yes
- Does there need to be a check-in in the future to determine whether to renew the data? (Yes/No) (If yes, set a todo reminder or file a bug if appropriate)**
Yes, will do a review of LP probes 9/2020 @cbonacuse
- Does the data collection use a third-party collection tool? If yes, escalate to legal.
Uses Leanplum, but that has been approved for use
@@ -101,6 +101,15 @@ class LeanplumMetricsService(private val application: Application) : MetricsServ | |||
|
|||
val installedApps = MozillaProductDetector.getInstalledMozillaProducts(application) | |||
|
|||
val trackingProtection = application.settings().run { | |||
when { | |||
!shouldUseTrackingProtection -> null |
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.
It may be fine, but I just want to make sure that Leanplum properly handles a "null" case here for when we data process this? I'm not tried adding a null value before.
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.
Renamed to String none
, and updated the docs.
0c2fb84
to
1db2335
Compare
Codecov Report
@@ Coverage Diff @@
## master #11965 +/- ##
============================================
+ Coverage 25.12% 25.14% +0.01%
Complexity 845 845
============================================
Files 389 389
Lines 15695 15703 +8
Branches 2020 2023 +3
============================================
+ Hits 3944 3949 +5
- Misses 11444 11447 +3
Partials 307 307
Continue to review full report at Codecov.
|
Fixes #11704: added tracking_protection_enabled attribute
Fixes #11657: add LP attribute for tracking protection
Pull Request checklist
[ ] Tests: This PR includes thorough tests or an explanation of why it does not[ ] Screenshots: This PR includes screenshots or GIFs of the changes made or an explanation of why it does not[ ] Accessibility: The code in this PR follows accessibility best practices or does not include any user facing features. In addition, it includes a screenshot of a successful accessibility scan to ensure no new defects are added to the product.After merge
To download an APK when reviewing a PR: