From mboxrd@z Thu Jan 1 00:00:00 1970 From: Michael Tremer To: development@lists.ipfire.org Subject: Re: [PATCH] Core Update 166: Drop orphaned convert-ovpn script Date: Mon, 21 Mar 2022 15:48:24 +0000 Message-ID: <5C5C3A62-2575-4F17-948D-87355F0199B2@ipfire.org> In-Reply-To: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============3600783712166967922==" List-Id: --===============3600783712166967922== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable Hello, Yes, I agree with the patch. A backup from that long ago should definitely create lots of other problems, = too. Should we create some mechanism that makes it impossible to restore a backup = file that is older than t0? -Michael > On 21 Mar 2022, at 13:30, Peter M=C3=BCller wr= ote: >=20 > This script only appeared in conjunction with Core Update 75, released > January 2014. Although it is still being executed while restoring a > backup, it would only be effective if anyone tried to restore a backup > created before C75. >=20 > I don't think there is a realistic need to carry this script along any > further. In doubt, it might be better to start from scratch again rather > than trying to restore an 8 year old backup, expecting everything to be > peachy and vanilla with it. >=20 > Signed-off-by: Peter M=C3=BCller > --- > config/backup/backup.pl | 5 +-- > config/rootfiles/common/aarch64/stage2 | 1 - > config/rootfiles/common/armv6l/stage2 | 1 - > config/rootfiles/common/x86_64/stage2 | 1 - > config/rootfiles/core/166/update.sh | 1 + > src/scripts/convert-ovpn | 60 -------------------------- > 6 files changed, 2 insertions(+), 67 deletions(-) > delete mode 100755 src/scripts/convert-ovpn >=20 > diff --git a/config/backup/backup.pl b/config/backup/backup.pl > index a2337cf23..1582789b6 100644 > --- a/config/backup/backup.pl > +++ b/config/backup/backup.pl > @@ -2,7 +2,7 @@ > ###########################################################################= #### > # = # > # IPFire.org - A linux based firewall = # > -# Copyright (C) 2007-2014 IPFire Team = # > +# Copyright (C) 2007-2022 IPFire Team = # > # = # > # This program is free software: you can redistribute it and/or modify = # > # it under the terms of the GNU General Public License as published by = # > @@ -153,9 +153,6 @@ restore_backup() { > # Reload firewall > firewallctrl >=20 > - # Convert old OpenVPN CCD files (CN change, Core Update 75) > - convert-ovpn > - > # Snort to suricata converter. > if [ -d "/var/ipfire/snort" ]; then > # Run converter > diff --git a/config/rootfiles/common/aarch64/stage2 b/config/rootfiles/comm= on/aarch64/stage2 > index e7eae625c..352c704d4 100644 > --- a/config/rootfiles/common/aarch64/stage2 > +++ b/config/rootfiles/common/aarch64/stage2 > @@ -92,7 +92,6 @@ usr/local/bin/backupiso > usr/local/bin/connscheduler > usr/local/bin/consort.sh > usr/local/bin/convert-dns-settings > -usr/local/bin/convert-ovpn > usr/local/bin/convert-to-location > usr/local/bin/filesystem-cleanup > usr/local/bin/hddshutdown > diff --git a/config/rootfiles/common/armv6l/stage2 b/config/rootfiles/commo= n/armv6l/stage2 > index 670a99927..198461a01 100644 > --- a/config/rootfiles/common/armv6l/stage2 > +++ b/config/rootfiles/common/armv6l/stage2 > @@ -90,7 +90,6 @@ usr/local/bin/backupiso > usr/local/bin/connscheduler > usr/local/bin/consort.sh > usr/local/bin/convert-dns-settings > -usr/local/bin/convert-ovpn > usr/local/bin/convert-to-location > usr/local/bin/filesystem-cleanup > usr/local/bin/hddshutdown > diff --git a/config/rootfiles/common/x86_64/stage2 b/config/rootfiles/commo= n/x86_64/stage2 > index d07c264b4..b03a7fecf 100644 > --- a/config/rootfiles/common/x86_64/stage2 > +++ b/config/rootfiles/common/x86_64/stage2 > @@ -92,7 +92,6 @@ usr/local/bin/backupiso > usr/local/bin/connscheduler > usr/local/bin/consort.sh > usr/local/bin/convert-dns-settings > -usr/local/bin/convert-ovpn > usr/local/bin/convert-to-location > usr/local/bin/filesystem-cleanup > usr/local/bin/hddshutdown > diff --git a/config/rootfiles/core/166/update.sh b/config/rootfiles/core/16= 6/update.sh > index 1370555b0..99bbe40f4 100644 > --- a/config/rootfiles/core/166/update.sh > +++ b/config/rootfiles/core/166/update.sh > @@ -37,6 +37,7 @@ rm -vf \ > /opt/pakfire/pakfire-2007.key \ > /usr/bin/mkinitrd \ > /usr/lib/dracut \ > + /usr/local/bin/convert-ovpn \ > /usr/local/bin/ovpn-ccd-convert \ > /usr/local/bin/rebuild-initrd >=20 > diff --git a/src/scripts/convert-ovpn b/src/scripts/convert-ovpn > deleted file mode 100755 > index 30e754326..000000000 > --- a/src/scripts/convert-ovpn > +++ /dev/null > @@ -1,60 +0,0 @@ > -#!/usr/bin/perl > - > -##########################################################################= ##### > -# = # > -# IPFire.org - A linux based firewall = # > -# Copyright (C) 2007-2022 IPFire Team = # > -# = # > -# This program is free software: you can redistribute it and/or modify = # > -# it under the terms of the GNU General Public License as published by = # > -# the Free Software Foundation, either version 3 of the License, or = # > -# (at your option) any later version. = # > -# = # > -# This program is distributed in the hope that it will be useful, = # > -# but WITHOUT ANY WARRANTY; without even the implied warranty of = # > -# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the = # > -# GNU General Public License for more details. = # > -# = # > -# You should have received a copy of the GNU General Public License = # > -# along with this program. If not, see . = # > -# = # > -##########################################################################= ##### > -# = # > -# This script converts old openvpn ccd files with underscore = # > -# to files with spaces to make them working with openvpn 2.3 again = # > -# STEP1: read ovpnconfig and verify cert names = # > -# STEP2: if neccessary convert ccd file = # > -# = # > -##########################################################################= ##### > - > -require '/var/ipfire/general-functions.pl'; > - > -my %configovpn=3D(); > -my $ccdpath=3D"/var/ipfire/ovpn/ccd/"; > -my $ovpnconfig=3D"/var/ipfire/ovpn/ovpnconfig"; > - > -&General::readhasharray ($ovpnconfig,\%configovpn); > - > -&check_config(); > - > -sub check_config { > - print "Converting CCD files...\n"; > - chdir($ccdpath); > - > - foreach my $key (sort keys %configovpn){ > - # Skip everything else but roadwarrior connections. > - next if ($configovpn{$key}[3] ne 'host'); > - > - # Skip all connections with no space in the CN name. > - next if ($configovpn{$key}[2] !~ " "); > - > - my $ccdname =3D $configovpn{$key}[2]; > - $ccdname =3D~ tr/ /_/; > - > - # Rename the CCD file if one with the old format exists. > - if (-e "$ccdname") { > - print " Renaming $ccdname -> $configovpn{$key}[2]...\n"; > - rename($ccdname, $configovpn{$key}[2]); > - } > - } > -} > --=20 > 2.34.1 --===============3600783712166967922==--