]> git.ipfire.org Git - ipfire-2.x.git/commit - html/cgi-bin/ovpnmain.cgi
ovpnmain.cgi: Fix for bug#11048 - insecure download icon shown for connections with...
authorAdolf Belka <adolf.belka@ipfire.org>
Mon, 25 Sep 2023 16:41:51 +0000 (18:41 +0200)
committerMichael Tremer <michael.tremer@ipfire.org>
Thu, 28 Sep 2023 09:33:21 +0000 (09:33 +0000)
commit8c5e71db2a53b3b443598a53dd01e43832234ca5
tree65ab8330d136bb7ada57593dcfd8cb00baa523c0
parente5ad33d9eeffbdeb1abbf2575853d880890ed228
ovpnmain.cgi: Fix for bug#11048 - insecure download icon shown for connections with a password

- At long last I have re-visited the patch submission for bug #11048 and fixed the issues
   that caused the problems last time I evaluated it in Testing.
- The insecure package download icon is shown if entry 41 in /var/ipfire/ovpn/ovpnconfig
   is set to no-pass. The code block on ovpnmain.cgi that deals with this checks if the
   connection is a host and if the first password entry is a null. Then it adds no-pass
   to ovpnconfig.
- The same block of code is also used for when he connection is edited. However at this
   stage the password entry is back to null because the password value is only kept until
   the connection has been saved. Therefore doing an edit results in the password value
   being taken as null even for connections with a password.
- This fix enters no-pass if the connection type is host and the password is null, pass if
   the connection type is host and the password has characters. If the connection type is
   net then no-pass is used as net2net connections dop not have encrypted certificates.
- The code has been changed to show a different icon for unencrypted and encrypted
   certificates.
- Separate patches are provided for the language file change, the provision of a new icon
   and the code for the update.sh script for the Core Update to update all existing
   connections, if any exist, to have either pass or no-pass in index 41.
- This patch set was a joint collaboration between Erik Kapfer and Adolf Belka
- Patch set, including the code for the Core Update 180 update.sh script has been tested
   on a vm testbed

Fixes: Bug#11048
Tested-by: Adolf Belka <adolf.belka@ipfire.org>
Suggested-by: Adolf Belka <adolf.belka@ipfire.org>
Suggested-by: Erik Kapfer <ummeegge@ipfire.org>
Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
html/cgi-bin/ovpnmain.cgi