Regression #13162
closed
Upgrade does not work when using only IPv6 DNS servers
Added by Sietse van Zanen over 2 years ago.
Updated over 2 years ago.
Plus Target Version:
22.05
Affected Architecture:
All
Description
When only IPv6 DNS servers are used (in general config) GUI update is not functional.
Following message is displayed when checking for updates in the GUI:
Retrieving: Unable to check for updates.
Unfortunately I have not been able to find any logs of this.
running pfSense-upgrade from cli works fine.
Adding one IPV4 dns server, no matter which order will make the GUI check functional.
- Related to Regression #11512: DHCP Leases page and ARP table page fail to load if DNS is not available added
- Tracker changed from Bug to Regression
- Related to Bug #12141: Lack of DNS or Internet connectivity causes GUI to be slow added
- Assignee set to Viktor Gurov
- Target version set to 2.7.0
- Plus Target Version set to 22.05
- Status changed from New to Pull Request Review
- Status changed from Pull Request Review to Feedback
- % Done changed from 0 to 100
- Subject changed from GUI Update not working when using only ipv6 DNS servers to Upgrade does not work when using only IPv6 DNS servers
Updating subject for release notes.
- Status changed from Feedback to Resolved
Seems to be fixed. On 22.01 if I set only IPv6 DNS and tell the GUI to only use remote DNS, the update check does fail.
Same setup on 22.05 works.
A couple of observations on this change, and the function in general. Firstly, there's a $nameservers variable being created by merging the hard coded test set with the system configured DNS servers, on line 2825 of system.inc. Either it should be iterated on in place of $gdns at line 2828, or it should be removed as useless code. Secondly, since almost all calls I could find to check_dnsavailable() do not pass any parameters, then the default for $proto should probably be 'any' and not 'inet', so that these checks still work on a system with only IPv6. Thirdly, it looks like on line 2833, the loop is limited to 3 iterations (ignoring the localhost skip), so the 4th entry of the 'any' array in the provided patch will never be used.
Also available in: Atom
PDF
Updated by Viktor Gurov 
Updated by 
Updated by