I did not merge this, as I believe we need this, because:
We won’t rewrite the OpenVPN configuration files on update, so it might be a good idea to just add the line and if someone edits the connection it might be removed.
That should work I believe and -legacy should not have any side effects when enabled but not needed.
Best, -Michael
On 7 Jun 2023, at 15:21, Adolf Belka adolf.belka@ipfire.org wrote:
- This code is no longer needed with the code in the ovpnmain.cgi patch in this patch set.
Tested-by: Adolf Belka adolf.belka@ipfire.org Signed-off-by: Adolf Belka adolf.belka@ipfire.org
config/backup/backup.pl | 15 --------------- 1 file changed, 15 deletions(-)
diff --git a/config/backup/backup.pl b/config/backup/backup.pl index 8d990c0f1..60138a58a 100644 --- a/config/backup/backup.pl +++ b/config/backup/backup.pl @@ -190,21 +190,6 @@ restore_backup() { # Update OpenVPN CRL /etc/fcron.daily/openvpn-crl-updater
- # Update OpenVPN N2N Client Configs
- ## Add providers legacy default line to n2n client config files
- # Check if ovpnconfig exists and is not empty
- if [ -s /var/ipfire/ovpn/ovpnconfig ]; then
# Identify all n2n connectionsfor y in $(awk -F',' '/net/ { print $3 }' /var/ipfire/ovpn/ovpnconfig); do# Add the legacy option to all N2N client conf files if it does not already exist- if [ $(grep -c "Open VPN Client Config" /var/ipfire/ovpn/n2nconf/${y}/${y}.conf) -eq 1 ] ; then
- if [ $(grep -c "providers legacy default" /var/ipfire/ovpn/n2nconf/${y}/${y}.conf) -eq 0 ] ; then
- echo "providers legacy default" >> /var/ipfire/ovpn/n2nconf/${y}/${y}.conf
- fi
- fi
done- fi
return 0 }
-- 2.40.1