deduplicate set_vpngateway_route

Daniel Lenskirequested to merge
fix_onlink_routes_and_deduplicate_set_vpngateway_route into master

Removes the unnecessary redundancy between set_vpngateway_route and set_vpngateway_route_attempt_reconnect, in favor of the latter.

While the Linux/iproute2 implementation of set_vpngateway_route_attempt_reconnect implementation is a bit more complex, it has some advantages:

  • Tries harder to find a non-looped-back route for the VPN gateway
  • Using ip route show captures some Linux kernel routing flags that ip route get apparently doesn't, including the onlink flag, which needs to be preserved to fix #20 (closed)

I've been using this updated version for several days, on multiple Linux/iproute2 systems, without any issues.

Edited by Daniel Lenski

Merge request reports