mozilla / doh-rollout

DEPRECATED - Add on for initial DoH rollout
Mozilla Public License 2.0
8 stars 4 forks source link

Telemetry event shows "evaluateReason": "userModified" instead of "evaluateReason": "netChange" #195

Closed dcicas closed 4 years ago

dcicas commented 4 years ago

[Affected Platforms]:

[Affected Versions]: Firefox Release 71.0 Firefox Beta 71.0b12 Firefox Nightly 72.0a1 (BuildID 20191127114233)

[Prerequisites]: Have the disable "DNSOverHTTPS" policy active. network.trr.mode is set to 5 The DoH add-on is installed.

{ "policies": { "DNSOverHTTPS": { "Enabled": false, "ProviderURL": "https://mozilla.cloudflare-dns.com/dns-query", "Locked": true } } }

[Steps to reproduce]:

  1. Open the browser with the profile from prerequisites.
  2. Disconnect from the internet.
  3. Reconnect to the internet.
  4. Navigate to about:telemetry and observe the events.
  5. Wait ~1min

[Expected result]:

[Actual result]:

[Notes]: The issue is reproducible with the 1.2.0rc2 version of the add-on.

nhi-nguyen commented 4 years ago

@dcicas is this issue fixed?

dcicas commented 4 years ago

image Hello @nhi-nguyen unfortunately this issue is not fixed using v1.2.0rc5

nhi-nguyen commented 4 years ago

Thanks for the update. This is a valid issue, however, I don't think it's a blocking issue. We will address this in the m-c version.

maxxcrawford commented 4 years ago

Bugzilla bug ported: https://bugzilla.mozilla.org/show_bug.cgi?id=1605004