1784258 - Add new telemetry probe for the DoH disablement reason

Status: RESOLVED FIXED

(Core :: Networking: DNS, task, P3)

Description

[Dragana Damjanovic [:dragana]]

Add a scalar telemetry probe with the following values:

• Opt-out
• User has manually changed prefs and TRR is disabled
• User has manually change prefs, and TRR is enabled via about:preferences
• Enterprise policy with the DNSOverHTTPS property set to false
• Enterprise policy without the DNSOverHTTPS property
• Heuristics trip
  • For each type, modify roots, third-party roots, global canary, Zscaler canary, os-level parental control, router-level parental control, VPNs, proxies and Name Resolution Policy Table, a separate value will be recorded. Multiple heuristics may be tripped at the same time and only the first one in this order will be recorded.
  • Heuristics may be tripped multiple times and only the first time heuristics has been detected the will be recorded, e.g. Each of the states above should have 2 different values:
    • TRR was disabled during the complete session
    • TRR was enabled and disable during the session

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1784258 - Remove unused UIOk from DoHController r=#necko

Comment 2

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1784258 - Add new telemetry probe for the DoH disablement reason r=#necko

This patch adds the following scalars:
networking.doh_heuristics_attempts - contains the number of heuristic attempts we performed during a subsession.
networking.doh_heuristics_pass_count - contains the number of times we passed heuristic during a subsession.
networking.doh_heuristics_result - the result of the last heuristic run.
networking.doh_heuristic_ever_tripped - a keyed scalar containing true if a heuristic was ever tripped during a subsession and false otherwise.

Depends on D187982

Comment 3

[Valentin Gosu [:valentin] (he/him)]

Attachment: request_1784258.md

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/0fd1a405cbf9
Remove unused UIOk from DoHController r=necko-reviewers,kershaw
https://hg.mozilla.org/integration/autoland/rev/76116e6c2476
Add new telemetry probe for the DoH disablement reason r=necko-reviewers,kershaw

Comment 5

[Narcis Beleuzu [:NarcisB]]

https://hg.mozilla.org/mozilla-central/rev/0fd1a405cbf9
https://hg.mozilla.org/mozilla-central/rev/76116e6c2476

Comment 6

[[e-mail]]

Comment on attachment 9352666 [details]
request_1784258.md

Is there or will there be documentation that describes the schema for the ultimate data set available publicly, complete and accurate?
​
Yes.
​
Is there a control mechanism that allows the user to turn the data collection on and off?
​
Yes. This collection can be controlled through Firefox's Preferences.
​
If the request is for permanent data collection, is there someone who will monitor the data over time?
​
Yes, Valentin Gosu is responsible.

Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?
​
Category 1, Technical.
​
Is the data collection request for default-on or default-off?
​
Default on for all channels.
​
Does the instrumentation include the addition of any new identifiers?
​
No.
​
Is the data collection covered by the existing Firefox privacy notice?
​
Yes.
​
Does the data collection use a third-party collection tool?
​
No.
​

Result: datareview+