1703216 - Get rid of network.trr.bootstrapAddress pref

Status: RESOLVED FIXED

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

Description

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

Bug 1700378 comment 11 is an example of a situation that has occurred more than once: people setting the bootstrapAddress pref then changing the provider URI.

In order to improve the situation here I propose the following:

• remove the network.trr.bootstrapAddress pref from all.js - this would make the pref inaccessible so users don't fiddle with it by mistake, and so it's ignored for users who already have it.
• Use a different pref name to achieve the same: network.trr.bootstrapAddr. We only use it in tests, and don't add it to all.js to make it less accessible.
• Optional follow-up: have a mechanism that resets the bootstrapAddr when changing the URI pref.

Comment 1

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

Attachment: Bug 1703216 - Get rid of network.trr.bootstrapAddress pref r=#necko,nhnt11

Depends on D109723

Comment 2

[(inactive) Nihanth Subramanya [:nhnt11]]

Should we consider another follow-up where we fall-back to native DNS for bootstrap if the user-set bootstrap address wasn't helpful?

Comment 3

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

I guess we could, but I'm not sure it's worth the complexity just yet.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/52be5dfc63af
Get rid of network.trr.bootstrapAddress pref r=nhnt11

Comment 5

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/52be5dfc63af