Closed Bug 1597683 Opened 10 months ago Closed 10 months ago

Parse DNS suffix list from registry

Categories

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

enhancement

Tracking

()

VERIFIED FIXED
mozilla72
Tracking Status
firefox71 --- wontfix
firefox72 --- verified

People

(Reporter: valentin, Assigned: valentin)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

In bug 1558840 we exposed the DNS suffix list on Windows. However, this only extracts the connection specific DNS suffix [1], not the search list you can also configure.
Initially I thought you can get the list via FirstDnsSuffix which is a linked list [2], but it seems that's not true. Instead, it seems it's only available from the windows registry.

[1] https://simpledns.com/kb/63/how-to-configure-dns-suffixes-on-windows-vista-2008
[2] https://docs.microsoft.com/en-us/windows/win32/api/iptypes/ns-iptypes-ip_adapter_addresses_lh

Pushed by valentin.gosu@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/a55bd3800cdf
Parse DNS suffix list from the windows registry r=mayhemer
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
See Also: → 1598575

This issue is verified fixed using Firefox 72.0a1 (BuildId:20191121214422) on Windows 10 64bit and Windows 7 64bit. Firefox parses the DNS suffix list from SYSTEM\CurrentControlSet\Services\Tcpip\Parameters successfully.

Valentin, is this a good candidate for a beta uplift? Since the code for Bug 1558840 is available in Beta 71?

Status: RESOLVED → VERIFIED
Flags: needinfo?(valentin.gosu)

(In reply to Emil Ghitta, QA [:emilghitta] from comment #4)

Valentin, is this a good candidate for a beta uplift? Since the code for Bug 1558840 is available in Beta 71?

Likely not, since the registry suffix list is less common than the per-network suffix. Also because of bug 1597137 comment 5 which might require backing out bug 1582472.

Flags: needinfo?(valentin.gosu)
You need to log in before you can comment on or make changes to this bug.