Activity log for bug #1592721

Date Who What changed Old value New value Message
2016-06-15 09:10:05 Mathieu Trudel-Lapierre bug added bug
2016-06-16 06:25:28 Mathieu Trudel-Lapierre network-manager (Ubuntu): status New In Progress
2016-06-16 06:25:30 Mathieu Trudel-Lapierre network-manager (Ubuntu): importance Undecided Medium
2016-06-16 06:25:32 Mathieu Trudel-Lapierre network-manager (Ubuntu): assignee Mathieu Trudel-Lapierre (cyphermox)
2016-06-16 11:11:47 Launchpad Janitor network-manager (Ubuntu): status In Progress Fix Released
2016-09-08 15:02:13 Mathieu Trudel-Lapierre description Currently, NM will write all search domains to both any DNS-handling plugins running, and also to resolv.conf / resolvconf; in all cases. The issue is that doing so means that in the split-DNS case on VPNs, you might get a negative response from all nameservers, then a new request by glibc with the search tacked on, to nameservers again, which might cause DNS requests for "private" resources (say, on the VPN) to be sent to external, untrusted resolvers, or for DNS queries not meant for VPN nameservers to be sent through the VPN anyway. This is fixable in the case where we have a caching plugin running (such as dnsmasq). dnsmasq will already know about the search domains and use that to limit queries to the right nameservers when a VPN is running. Writing search domains to resolv.conf is unnecessary in this case. We should still write search domains if no caching gets done, as we then need to expect glibc to send requests as it otherwise would. [Impact] All VPN users meaning to use split-tunnelling in a situation where leaking DNS data to the internet about what might be behind their VPN is undesirable. [Test case] 1) connect to VPN 2) Use dig to request a name that should be on the VPN 3) Verify (kill -USR1 the dnsmasq binary from NM) that the request has only gone through the VPN nameservers (only its request number should have increased by one) 4) Use dig to request a name off-VPN, such as google.com. 5) Verify (kill -USR1 again) that the request has caused the non-VPN nameserver request number to increase, and that the VPN number has not increased. It is easier to verify this when there is as little traffic as possible on the system, to avoid spurious DNS requests which would make it harder to validate the counters. [Regression potential] This change modifies the way in which DNS nameservers and search domains are passed to dnsmasq, as such, if a VPN is configured in a non-standard way and intended to be used to resolve all network requests (which is typically not the case for split-tunnelling) or if the public network is intended to always resolve all requests while the VPN still provides search domains, one might observe incorrect behavior. Possible failure cases would include failure to resolve names correctly (resulting in NXDOMAIN or REFUSED from dnsmasq) or resolving to the incorrect values (if the wrong nameserver is used). --- Currently, NM will write all search domains to both any DNS-handling plugins running, and also to resolv.conf / resolvconf; in all cases. The issue is that doing so means that in the split-DNS case on VPNs, you might get a negative response from all nameservers, then a new request by glibc with the search tacked on, to nameservers again, which might cause DNS requests for "private" resources (say, on the VPN) to be sent to external, untrusted resolvers, or for DNS queries not meant for VPN nameservers to be sent through the VPN anyway. This is fixable in the case where we have a caching plugin running (such as dnsmasq). dnsmasq will already know about the search domains and use that to limit queries to the right nameservers when a VPN is running. Writing search domains to resolv.conf is unnecessary in this case. We should still write search domains if no caching gets done, as we then need to expect glibc to send requests as it otherwise would.
2016-09-08 15:02:40 Mathieu Trudel-Lapierre nominated for series Ubuntu Xenial
2016-09-08 15:02:40 Mathieu Trudel-Lapierre bug task added network-manager (Ubuntu Xenial)
2016-09-08 15:03:30 Mathieu Trudel-Lapierre network-manager (Ubuntu Xenial): status New In Progress
2016-09-08 15:03:32 Mathieu Trudel-Lapierre network-manager (Ubuntu Xenial): importance Undecided Medium
2016-09-08 15:03:34 Mathieu Trudel-Lapierre network-manager (Ubuntu Xenial): assignee Mathieu Trudel-Lapierre (cyphermox)
2016-09-27 09:29:44 Andy Whitcroft network-manager (Ubuntu Xenial): status In Progress Fix Committed
2016-09-27 09:29:47 Andy Whitcroft bug added subscriber Ubuntu Stable Release Updates Team
2016-09-27 09:29:51 Andy Whitcroft bug added subscriber SRU Verification
2016-09-27 09:29:58 Andy Whitcroft tags verification-needed
2016-10-12 08:23:18 Launchpad Janitor network-manager (Ubuntu Xenial): status Fix Committed Fix Released