Closed Bug 1470005 Opened 2 years ago Closed 2 years ago

TRR: make blacklist duration one minute by default

Categories

(Core :: Networking: DNS, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
mozilla62
Tracking Status
firefox62 --- fixed

People

(Reporter: bagder, Assigned: bagder)

Details

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

Attachments

(1 file)

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 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+
Pushed by daniel@haxx.se:
https://hg.mozilla.org/integration/autoland/rev/e09fbaee1456
change default TRR blacklist duration to 60 seconds r=mcmanus
https://hg.mozilla.org/mozilla-central/rev/e09fbaee1456
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla62
You need to log in before you can comment on or make changes to this bug.