Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DNS66 reverses the order of the system DNS servers #382

Closed
Platway opened this issue Feb 20, 2020 · 3 comments
Closed

DNS66 reverses the order of the system DNS servers #382

Platway opened this issue Feb 20, 2020 · 3 comments
Labels
bug help wanted moreinfo More information is needed

Comments

@Platway
Copy link

Platway commented Feb 20, 2020

When my Android phone connects to my home WiFi, the primary DNS server is 192.168.1.1, and the secondary is 8.8.8.8. "Custom DNS servers" are disabled in DNS66.

According to the logcat, DNS66 uses 8.8.8.8 as DNS server prior to 192.168.1.1. This is unreasonable. Why does DNS66 reverses the order of the system DNS servers?

DNS66 version is 0.6.1.

@julian-klode
Copy link
Owner

It does not. It creates one virtual dns server for each system DNS server, it's up to Android to pick one. Android is fairly quick to switch it's DNS servers.

@julian-klode
Copy link
Owner

Yuck, so Android seems to have switched the order in which DNS servers are returned, I'm not sure what can be done here, but this needs more investigation.

02-28 20:58:01.080 10397 20220 I AdVpnThread: Got DNS servers = [/8.8.8.8, /2a02:908:2813:4180:d63f:cbff:fe85:ee4, /192.168.0.1]
02-28 20:58:01.080 10397 20220 D AdVpnThread: configure: Adding IPv6 address/2001:db8::
02-28 20:58:01.081 10397 20220 I AdVpnThread: configure: Adding DNS Server /8.8.8.8 as 192.0.2.2
02-28 20:58:01.081 10397 20220 I AdVpnThread: configure: Adding DNS Server /2a02:908:2813:4180:d63f:cbff:fe85:ee4 as /2001:db8::3
02-28 20:58:01.081 10397 20220 I AdVpnThread: configure: Adding DNS Server /192.168.0.1 as 192.0.2.4

@julian-klode julian-klode reopened this Feb 28, 2020
@julian-klode julian-klode added bug help wanted moreinfo More information is needed labels Feb 28, 2020
@julian-klode
Copy link
Owner

Ah yikes, their in a HashSet, that's not ordered

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug help wanted moreinfo More information is needed
Projects
None yet
Development

No branches or pull requests

2 participants