Closed Bug 1466860 Opened 4 years ago Closed 4 years ago

TRR: update to latest DOH draft version


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




Tracking Status
firefox62 --- fixed
firefox63 --- fixed


(Reporter: bagder, Assigned: bagder)


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


(1 file)

Currently Firefox runs a version of the DNS-over-HTTPS protocol that matches the -04 version of the draft ( but there have been several changes to it since.


 - The query string used in GET requests
 - The Content-Type set in requests
 - The accepted Content-Type in responses

Needs to take Bug 1446404 into consideration and also the implementation state of the end-points existing users are most likely using.
MozReview-Commit-ID: FSVh7qUbBig
Server interop status

Cloudflare: check
Google: nope (awaiting answer on specifics)
Comment on attachment 8998111 [details]
bug 1466860 - upgrade DOH code to draft-12 level

Patrick McManus [:mcmanus] has approved the revision.
Attachment #8998111 - Flags: review+
Pushed by
upgrade DOH code to draft-12 level r=mcmanus
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Is this something that can ride the trains or does it need a Beta uplift request?
Flags: needinfo?(daniel)
Comment on attachment 8998111 [details]
bug 1466860 - upgrade DOH code to draft-12 level

Thanks for asking. I'm requesting an uplift of this to keep the DoH-support in 62 at fine level.

Approval Request Comment
[Feature/Bug causing the regression]:
[User impact if declined]: This patch ups Firefox' DoH support to the latest spec draft, which is the one likely to ship as RFC soon. The wire format is not expected to change anything further.
[Is this code covered by automated tests?]: Yes, and this patch updates the test.
[Has the fix been verified in Nightly?]: Yes
[Needs manual test from QE? If yes, steps to reproduce]: 
[List of other uplifts needed for the feature/fix]:
[Is the change risky?]: no
[Why is the change risky/not risky?]: The change is simple and there are very few users of DoH on 62, and they've all opted in via manual prefs edits.
[String changes made/needed]: none
Flags: needinfo?(daniel)
Attachment #8998111 - Flags: approval-mozilla-beta?
Comment on attachment 8998111 [details]
bug 1466860 - upgrade DOH code to draft-12 level

Keeping 62 opt-in users to the newest spec sounds good to me. 
Let's uplift for beta 18.
Attachment #8998111 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
You need to log in before you can comment on or make changes to this bug.