From: Michael Tremer <michael.tremer@ipfire.org>
Date: Tue, 8 Jul 2025 08:38:44 +0000 (+0200)
Subject: Revert "ovpnmain.cgi: Remove yet another "if (1)" statement"
X-Git-Tag: v2.29-core197~25^2~74
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=eed199788837a54f739d567afe225abe02b0012f;p=ipfire-2.x.git

Revert "ovpnmain.cgi: Remove yet another "if (1)" statement"

This reverts commit 0dcafefb694d4e1ebef317f4d45f68216685ff25.

Removing this breaks creating N2N connections and I don't think there is
a way to fix this all properly without a major rewrite.

Signed-off-by: Michael Tremer <michael.tremer@ipfire.org>
---

diff --git a/html/cgi-bin/ovpnmain.cgi b/html/cgi-bin/ovpnmain.cgi
index 58a321175..6e9c64aa7 100644
--- a/html/cgi-bin/ovpnmain.cgi
+++ b/html/cgi-bin/ovpnmain.cgi
@@ -4523,6 +4523,7 @@ if ($cgiparams{'TYPE'} eq 'net') {
     $checked{'TLSAUTH'}{'on'} = '';
     $checked{'TLSAUTH'}{$cgiparams{'TLSAUTH'}} = 'CHECKED';
 
+    if (1) {
 	&Header::showhttpheaders();
 	&Header::openpage($Lang::tr{'ovpn'}, 1, '');
 	&Header::openbigbox('100%', 'LEFT', '', $errormessage);
@@ -4761,6 +4762,7 @@ if ($cgiparams{'TYPE'} eq 'host') {
 		}
 		print "</table><br><br><hr><br><br>";
 	}
+}
 
 	&Header::closebox();
 	if ($cgiparams{'KEY'} && $cgiparams{'AUTH'} eq 'psk') {