TRR: make blacklist duration one minute by default

RESOLVED FIXED in Firefox 62

Status

()

enhancement
P1
normal
RESOLVED FIXED
Last year
Last year

People

(Reporter: bagder, Assigned: bagder)

Tracking

unspecified
mozilla62
Points:
---

Firefox Tracking Flags

(firefox62 fixed)

Details

(Whiteboard: [necko-triaged][trr])

Attachments

(1 attachment)

Assignee

Description

Last year
1. The normal back-off + retry mechanism works very well and is transparent enough to not make users hurt much even when not blacklisted
2. This removes "mistaken" blacklist entries faster
3. The new expiration timeout is set to mostly survive a page load and a little more.
Comment hidden (mozreview-request)

Comment 2

Last year
mozreview-review
Comment on attachment 8986590 [details]
bug 1470005 - change default TRR blacklist duration to 60 seconds

https://reviewboard.mozilla.org/r/251878/#review258336

::: modules/libpref/init/all.js:5303
(Diff revision 1)
>  // from the DOH end point to ensure proper configuration
>  pref("network.trr.confirmationNS", "example.com");
>  // hardcode the resolution of the hostname in network.trr.uri instead of
>  // relying on the system resolver to do it for you
>  pref("network.trr.bootstrapAddress", "");
> -// TRR blacklist entry expire time (in seconds). Default is 20 minutes.
> +// TRR blacklist entry expire time (in seconds). Default is one minute.

adjust the comment to indicate we're just looking for locality around a page load
Attachment #8986590 - Flags: review?(mcmanus) → review+
Comment hidden (mozreview-request)

Comment 4

Last year
Pushed by daniel@haxx.se:
https://hg.mozilla.org/integration/autoland/rev/e09fbaee1456
change default TRR blacklist duration to 60 seconds r=mcmanus

Comment 5

Last year
bugherder
https://hg.mozilla.org/mozilla-central/rev/e09fbaee1456
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.