From mboxrd@z Thu Jan  1 00:00:00 1970
From: Adolf Belka <adolf.belka@ipfire.org>
To: development@lists.ipfire.org
Subject: [PATCH v3 7/7] optionsfw.cgi: Move Firewall Options Drop commands to
 before the logging section
Date: Sun, 21 Jan 2024 12:45:53 +0100
Message-ID: <20240121114553.5182-7-adolf.belka@ipfire.org>
In-Reply-To: <20240121114553.5182-1-adolf.belka@ipfire.org>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4626852105114128167=="
List-Id: <development.lists.ipfire.org>

--===============4626852105114128167==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: quoted-printable

- Moved the Firewall Options Drop commands to before the logging section, as =
discussed
   at January 2024 Video Call.

Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
---
 html/cgi-bin/optionsfw.cgi | 47 +++++++++++++++++++-------------------
 1 file changed, 24 insertions(+), 23 deletions(-)

diff --git a/html/cgi-bin/optionsfw.cgi b/html/cgi-bin/optionsfw.cgi
index 52ac1b01e..60b1bdd91 100644
--- a/html/cgi-bin/optionsfw.cgi
+++ b/html/cgi-bin/optionsfw.cgi
@@ -2,7 +2,7 @@
 ############################################################################=
###
 #                                                                           =
  #
 # IPFire.org - A linux based firewall                                       =
  #
-# Copyright (C) 2007-2022  IPFire Team  <info(a)ipfire.org>                 =
    #
+# Copyright (C) 2007-2024  IPFire Team  <info(a)ipfire.org>                 =
    #
 #                                                                           =
  #
 # 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      =
  #
@@ -224,6 +224,29 @@ END
=20
 	<br>
=20
+<table width=3D'95%' cellspacing=3D'0'>
+	<tr bgcolor=3D'$color{'color20'}'>
+		<td colspan=3D'2' align=3D'left'><b>$Lang::tr{'fw red'}</b></td>
+	</tr>
+	<tr>
+		<td align=3D'left' width=3D'60%'>$Lang::tr{'drop hostile'}</td>
+		<td align=3D'left'>
+			$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPHOSTILE' value=3D'on' $=
checked{'DROPHOSTILE'}{'on'} />/
+			<input type=3D'radio' name=3D'DROPHOSTILE' value=3D'off' $checked{'DROPHO=
STILE'}{'off'} /> $Lang::tr{'off'}
+		</td>
+	</tr>
+</table>
+<br>
+
+<table width=3D'95%' cellspacing=3D'0'>
+<tr bgcolor=3D'$color{'color20'}'><td colspan=3D'2' align=3D'left'><b>$Lang:=
:tr{'fw blue'}</b></td></tr>
+<tr><td align=3D'left' width=3D'60%'>$Lang::tr{'drop proxy'}</td><td align=
=3D'left'>$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPPROXY' value=3D'o=
n' $checked{'DROPPROXY'}{'on'} />/
+																						<input type=3D'radio' name=3D'DROPPROXY' value=3D'off'=
 $checked{'DROPPROXY'}{'off'} /> $Lang::tr{'off'}</td></tr>
+<tr><td align=3D'left' width=3D'60%'>$Lang::tr{'drop samba'}</td><td align=
=3D'left'>$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPSAMBA' value=3D'o=
n' $checked{'DROPSAMBA'}{'on'} />/
+																						<input type=3D'radio' name=3D'DROPSAMBA' value=3D'off'=
 $checked{'DROPSAMBA'}{'off'} /> $Lang::tr{'off'}</td></tr>
+</table>
+<br>
+
 <table width=3D'95%' cellspacing=3D'0'>
 	<tr bgcolor=3D'$color{'color20'}'>
 		<td colspan=3D'2' align=3D'left'><b>$Lang::tr{'fw logging'}</b></td>
@@ -308,28 +331,6 @@ END
 </table>
 <br/>
=20
-<table width=3D'95%' cellspacing=3D'0'>
-	<tr bgcolor=3D'$color{'color20'}'>
-		<td colspan=3D'2' align=3D'left'><b>$Lang::tr{'fw red'}</b></td>
-	</tr>
-	<tr>
-		<td align=3D'left' width=3D'60%'>$Lang::tr{'drop hostile'}</td>
-		<td align=3D'left'>
-			$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPHOSTILE' value=3D'on' $=
checked{'DROPHOSTILE'}{'on'} />/
-			<input type=3D'radio' name=3D'DROPHOSTILE' value=3D'off' $checked{'DROPHO=
STILE'}{'off'} /> $Lang::tr{'off'}
-		</td>
-	</tr>
-</table>
-<br>
-
-<table width=3D'95%' cellspacing=3D'0'>
-<tr bgcolor=3D'$color{'color20'}'><td colspan=3D'2' align=3D'left'><b>$Lang:=
:tr{'fw blue'}</b></td></tr>
-<tr><td align=3D'left' width=3D'60%'>$Lang::tr{'drop proxy'}</td><td align=
=3D'left'>$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPPROXY' value=3D'o=
n' $checked{'DROPPROXY'}{'on'} />/
-																						<input type=3D'radio' name=3D'DROPPROXY' value=3D'off'=
 $checked{'DROPPROXY'}{'off'} /> $Lang::tr{'off'}</td></tr>
-<tr><td align=3D'left' width=3D'60%'>$Lang::tr{'drop samba'}</td><td align=
=3D'left'>$Lang::tr{'on'} <input type=3D'radio' name=3D'DROPSAMBA' value=3D'o=
n' $checked{'DROPSAMBA'}{'on'} />/
-																						<input type=3D'radio' name=3D'DROPSAMBA' value=3D'off'=
 $checked{'DROPSAMBA'}{'off'} /> $Lang::tr{'off'}</td></tr>
-</table>
-<br>
 <table width=3D'95%' cellspacing=3D'0'>
 <tr bgcolor=3D'$color{'color20'}'><td colspan=3D'2' align=3D'left'><b>$Lang:=
:tr{'fw settings'}</b></td></tr>
 <tr><td align=3D'left' width=3D'60%'>$Lang::tr{'fw settings color'}</td><td =
align=3D'left'>$Lang::tr{'on'} <input type=3D'radio' name=3D'SHOWCOLORS' valu=
e=3D'on' $checked{'SHOWCOLORS'}{'on'} />/
--=20
2.43.0


--===============4626852105114128167==--