After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 774721 - [review] lr/ipv6: support IPv6-only connections
[review] lr/ipv6: support IPv6-only connections
Status: RESOLVED FIXED
Product: NetworkManager
Classification: Platform
Component: VPN: openconnect
git master
Other Linux
: Normal normal
: ---
Assigned To: David Woodhouse
NetworkManager maintainer(s)
Depends on:
Blocks:
 
 
Reported: 2016-11-19 14:49 UTC by Lubomir Rintel
Modified: 2016-11-30 08:11 UTC
See Also:
GNOME target: ---
GNOME version: ---


Description Lubomir Rintel 2016-11-19 14:49:42 UTC 
$subj says it all

https://git.gnome.org/browse/network-manager-openconnect/log/?h=lr/ipv6
Comment 1 Thomas Haller 2016-11-20 11:56:16 UTC 
lgtm (not tested)
Comment 2 David Woodhouse 2016-11-24 11:42:44 UTC 
I don't like the last commit. DNS is *not* a per-IP-protocol thing. You don't use *different* DNS servers for IPv6 vs. Legacy IP. But that's more a problem in NetworkManager itself. Like NIS and other things, DNS ought to be *generic* connection config, not part of the IP4 config or the IP6 config. So in this specific context I suppose there's not a lot you can do; I just wanted to whine about it (again).

As long as this is tested (and given that we have ocserv, why wouldn't it be?), feel free to push it. Thanks.
Comment 3 Lubomir Rintel 2016-11-24 12:28:35 UTC 
(In reply to David Woodhouse from comment #2)
> I don't like the last commit. DNS is *not* a per-IP-protocol thing. You
> don't use *different* DNS servers for IPv6 vs. Legacy IP. But that's more a
> problem in NetworkManager itself. Like NIS and other things, DNS ought to be
> *generic* connection config, not part of the IP4 config or the IP6 config.
> So in this specific context I suppose there's not a lot you can do; I just
> wanted to whine about it (again).
> 
> As long as this is tested (and given that we have ocserv, why wouldn't it
> be?), feel free to push it. Thanks.

Yes, it has been tested with ocserv.