ovpnmain.cgi: Fixes bug#13190 - connection status shows disconnected for connected client

Message ID 20230727135725.7053-1-adolf.belka@ipfire.org
State Staged
Commit f46e8298ad63e3fa559dd9d9d7981e224cc5a64b
Headers
Series ovpnmain.cgi: Fixes bug#13190 - connection status shows disconnected for connected client |

Commit Message

Adolf Belka July 27, 2023, 1:57 p.m. UTC
  - If the certificate name has underscores in it then the status always shows as DISCONNECTED
   alothough the actual connection is working and can be used.
- The certificate with underscores works fine. RFC5280 accepts underscores in the name.
- The code for checking the status splits up the status message and takes the first part
   as the common name for the connection. Then there is a regex command which rerplaces
   any underscores in the status common name with spaces. This results in the connection
   with underscores in the certificate name never matching any status feedback common
   name as the underscores have been replaced by spaces.
- This has been tested to work with my vm test bed. With existing code the connection with
   underscores in the certificate name permanently showed DISCONNECTED. With the code change
   the connection shows as CONNECTED very quickly.

Fixes: Bug#13190
Tested-by: Adolf Belka <adolf.belka@ipfire.org>
Signed-off-by: Adolf Belka <adolf.belka@ipfire.org>
---
 html/cgi-bin/ovpnmain.cgi | 1 -
 1 file changed, 1 deletion(-)
  

Patch

diff --git a/html/cgi-bin/ovpnmain.cgi b/html/cgi-bin/ovpnmain.cgi
index f734331fa..5afe54f55 100755
--- a/html/cgi-bin/ovpnmain.cgi
+++ b/html/cgi-bin/ovpnmain.cgi
@@ -5502,7 +5502,6 @@  END
 				if ($match[1] ne "Common Name") {
 					$cn = $match[1];
 				}
-				$cn =~ s/[_]/ /g;
 				if ($cn eq "$confighash{$key}[2]") {
 					$col1="bgcolor='${Header::colourgreen}'";
 					$active = "<b><font color='#FFFFFF'>$Lang::tr{'capsopen'}</font></b>";