Skip to content

Allow any kind of VPN as secondaries (Fixed #1395)

Summary

As of now, NM allows only connections derived from the VPN plugin to be used as secondaries. This excludes non-VPN-plugin connections still providing VPN functionality, such as Wireguard, to be used as secondary. This commit extends NetworkManager to allow any kind of VPN connection to be used as secondary.

Purpose

Treat Wireguard also as VPN.

Checklist

Please read https://gitlab.freedesktop.org/NetworkManager/NetworkManager/-/blob/main/CONTRIBUTING.md before opening the merge request. In particular, check that:

  • the subject for all commits is concise and explicative
  • the message for all commits explains the reason for the change
  • the source is properly formatted
  • any relevant documentation is up to date
  • you have added unit tests if applicable

Notes

This is my first attempt. I'm still trying to understand NM, and looking for an optimal solution. nm_connection_is_vpn_a() is just a temp name until I know a better one.

@thaller @ihuguet Any thoughts on the direction this is going?

Edited by Alexander Lochmann

Merge request reports