net/dns: don't try to configure LLMNR or mdns in NetworkManager.
Fixes #1870. Signed-off-by: David Anderson <danderson@tailscale.com>
This commit is contained in:
parent
e78e26b6fb
commit
77e2375501
|
@ -175,9 +175,12 @@ func (m *nmManager) trySet(ctx context.Context, config OSConfig) error {
|
||||||
search = append(search, "~.")
|
search = append(search, "~.")
|
||||||
}
|
}
|
||||||
|
|
||||||
general := settings["connection"]
|
// Ideally we would like to disable LLMNR and mdns on the
|
||||||
general["llmnr"] = dbus.MakeVariant(0)
|
// interface here, but older NetworkManagers don't understand
|
||||||
general["mdns"] = dbus.MakeVariant(0)
|
// those settings and choke on them, so we don't. Both LLMNR and
|
||||||
|
// mdns will fail since tailscale0 doesn't do multicast, so it's
|
||||||
|
// effectively fine. We used to try and enforce LLMNR and mdns
|
||||||
|
// settings here, but that led to #1870.
|
||||||
|
|
||||||
ipv4Map := settings["ipv4"]
|
ipv4Map := settings["ipv4"]
|
||||||
ipv4Map["dns"] = dbus.MakeVariant(dnsv4)
|
ipv4Map["dns"] = dbus.MakeVariant(dnsv4)
|
||||||
|
|
Loading…
Reference in New Issue