Closed Bug 1466860 Opened 2 years ago Closed 2 years ago
TRR: update to latest DOH draft version
Currently Firefox runs a version of the DNS-over-HTTPS protocol that matches the -04 version of the draft (https://tools.ietf.org/html/draft-ietf-doh-dns-over-https-04) but there have been several changes to it since. Including: - 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.
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. https://phabricator.services.mozilla.com/D2828
Attachment #8998111 - Flags: review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/b493b05dcbd5 upgrade DOH code to draft-12 level r=mcmanus
Is this something that can ride the trains or does it need a Beta uplift request?
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
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.