-
Notifications
You must be signed in to change notification settings - Fork 87
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
Split DNS check incorrectly uses ResolveConfSplitDNSProvider on Fedora 37 #129
Comments
Is this issued fixed by #126? |
Seems like using NetworkManager /etc/resolv.conf is not a symlink and NetworkManager manages the file. |
According to the systemd-resolved documentation:
As far as I know, Fedora 37 desktops are based on systemd-resolved underneath NetworkManager, in which case |
I guess I commented in the wrong issue. I use Archlinux with NetworkManager, I probably should have put my comment in #124 , since that is the exact error I get. |
Indeed, Arch Linux is not based on systemd-resolved, and |
Also see #118 (comment). |
That outputs
That path seems to be the same, so hopefully the fix is also the same 👍 Apologies for the slow reply, seems I missed the original notifications. |
A bit of background:
ResolveConfSplitDNSProvider interacts with the DNS resolver using
resolvconf
, while ResolvedSplitDNSProvider usesresolvectl
. vpn-slice decides which of these to use by inspecting/etc/resolv.conf
.If I understand things correclty, it looks for the string "/run/systemd/resolve/" in /etc/resolv.conf, which is not present on Fedora 37. Instead, it might be better to check whether resolvectl exists on the system?
Using resolvctl allows me to make the split DNS a bit more splitty with the following change (the last line is the only change).
This disables the default-route flag from the tunneled interface, which means it won't be used to DNS queries UNLESS it's in the
domains
list.Without this change, every DNS query is sent to the tunneled resolver, even if another resolver has already answered the query.
I'd submit this as a merge request, but I don't understand the code well enough to make sure that this won't break anyone else's setup. Feel free to reach out for more info.
The text was updated successfully, but these errors were encountered: