]> git.ipfire.org Git - people/mfischer/ipfire-2.x.git/commit
vpnmain.cgi: Fix for 2nd part of bug10595
authorAdolf Belka <adolf.belka@ipfire.org>
Thu, 6 Mar 2025 11:32:20 +0000 (12:32 +0100)
committerMichael Tremer <michael.tremer@ipfire.org>
Mon, 10 Mar 2025 09:59:23 +0000 (09:59 +0000)
commitf82c1bd187d7a3a1001db4bb42b3f989f9c223f7
tree7bde3a4f0c10da759fc9455eb66ccdf97f468af7
parent0c604256205949ce255ac8127e98dcb333ecee04
vpnmain.cgi: Fix for 2nd part of bug10595

- Bug10595 had two parts in it and was closed after the first part was fixed. The second
   part was still unfixed at that time. I cam across it when checking out an open bug on
   a similar issue with OpenVPN.
- I found the section that checks on the CA Name and modified it to also allow spaces.
- Having modified that then the subroutines getsubjectfromcert and getCNfromcert required
   modifications otherwise the openssl statement only got a filename with the first
   portion of the ca name until the first space was encountered. This v2 version of this
   patch set has the safe approach suggested by @Michael. This v3 version has been
   re based to another patch submission that modified lines in a similar place and
   prevented a merge to work.
- I am open to any suggestions for improvements to how I implemented the use of the
   &General::system_output function
- Tested this change out on my vm and it worked fine. I was able to upload a ca
   certificate into IPSec and use spaces in the CA Name.
- Changed the test for the CA_NAME to allow spaces. Change also made to en.pl file

Fixes: Bug10595 part 2
Tested-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
html/cgi-bin/vpnmain.cgi