[v2] netother.cgi: show content of routing table 220

Message ID f3812fbf-f554-c7be-7c3a-54bf71c86d65@ipfire.org
State Accepted
Headers
Series [v2] netother.cgi: show content of routing table 220 |

Commit Message

Peter Müller March 21, 2020, 4:03 p.m. UTC
  Since IPsec routing information do not show up in the normal routing
table, also displaying the contents of table 220 on netother.cgi might
be useful for debugging purposes.

The second version of this patch omits the output if routing table 220
is empty and introduces a custom translation for IPsec routing table
entries instead of just adding the table number to the generic translation.

Cc: Michael Tremer <michael.tremer@ipfire.org>
Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
---
 html/cgi-bin/netother.cgi | 8 ++++++++
 langs/de/cgi-bin/de.pl    | 1 +
 langs/en/cgi-bin/en.pl    | 1 +
 3 files changed, 10 insertions(+)
 mode change 100644 => 100755 html/cgi-bin/netother.cgi
  

Comments

Michael Tremer March 22, 2020, 9:24 a.m. UTC | #1
Reviewed-by: Michael Tremer <michael.tremer@ipfire.org>

> On 21 Mar 2020, at 16:03, Peter Müller <peter.mueller@ipfire.org> wrote:
> 
> Since IPsec routing information do not show up in the normal routing
> table, also displaying the contents of table 220 on netother.cgi might
> be useful for debugging purposes.
> 
> The second version of this patch omits the output if routing table 220
> is empty and introduces a custom translation for IPsec routing table
> entries instead of just adding the table number to the generic translation.
> 
> Cc: Michael Tremer <michael.tremer@ipfire.org>
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
> html/cgi-bin/netother.cgi | 8 ++++++++
> langs/de/cgi-bin/de.pl    | 1 +
> langs/en/cgi-bin/en.pl    | 1 +
> 3 files changed, 10 insertions(+)
> mode change 100644 => 100755 html/cgi-bin/netother.cgi
> 
> diff --git a/html/cgi-bin/netother.cgi b/html/cgi-bin/netother.cgi
> old mode 100644
> new mode 100755
> index dde1b603a..4f03c82e8
> --- a/html/cgi-bin/netother.cgi
> +++ b/html/cgi-bin/netother.cgi
> @@ -79,6 +79,14 @@ if ( $querry[0] =~ "fwhits"){
> 	print "<pre>$output</pre>\n";
> 	&Header::closebox();
> 
> +	$output = `/sbin/ip route list table 220`;
> +	if ( $output ) {
> +		&Header::openbox('100%', 'left', $Lang::tr{'ipsec routing table entries'});
> +		$output = &Header::cleanhtml($output,"y");
> +		print "<pre>$output</pre>\n";
> +		&Header::closebox()
> +	}
> +
> 	&Header::openbox('100%', 'left', $Lang::tr{'arp table entries'});
> 	$output = `/sbin/ip neigh show`;
> 	$output = &Header::cleanhtml($output,"y");
> diff --git a/langs/de/cgi-bin/de.pl b/langs/de/cgi-bin/de.pl
> index 80579e7cc..6f7b4db59 100644
> --- a/langs/de/cgi-bin/de.pl
> +++ b/langs/de/cgi-bin/de.pl
> @@ -1498,6 +1498,7 @@
> 'ipsec mode tunnel' => 'Tunnel',
> 'ipsec network' => 'IPsec-Netzwerk',
> 'ipsec no connections' => 'Keine aktiven IPsec-Verbindungen',
> +'ipsec routing table entries' => 'Einträge der IPsec-Routing-Tabelle',
> 'ipsec settings' => 'IPsec-Einstellungen',
> 'iptable rules' => 'IPTable-Regeln',
> 'iptmangles' => 'IPTable Mangles',
> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
> index a68c8f411..1e1aed53c 100644
> --- a/langs/en/cgi-bin/en.pl
> +++ b/langs/en/cgi-bin/en.pl
> @@ -1547,6 +1547,7 @@
> 'ipsec mode tunnel' => 'Tunnel',
> 'ipsec network' => 'IPsec network',
> 'ipsec no connections' => 'No active IPsec connections',
> +'ipsec routing table entries' => 'IPsec Routing Table Entries',
> 'ipsec settings' => 'IPsec Settings',
> 'iptable rules' => 'IPTable rules',
> 'iptmangles' => 'IPTable Mangles',
> -- 
> 2.16.4
>
  
Arne Fitzenreiter March 30, 2020, 5:38 p.m. UTC | #2
This patch doesn't apply. It also change the rights of the netother.cgi 
in git. This should be 644 not 755


Am 2020-03-21 17:03, schrieb Peter Müller:
> Since IPsec routing information do not show up in the normal routing
> table, also displaying the contents of table 220 on netother.cgi might
> be useful for debugging purposes.
> 
> The second version of this patch omits the output if routing table 220
> is empty and introduces a custom translation for IPsec routing table
> entries instead of just adding the table number to the generic 
> translation.
> 
> Cc: Michael Tremer <michael.tremer@ipfire.org>
> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
> ---
>  html/cgi-bin/netother.cgi | 8 ++++++++
>  langs/de/cgi-bin/de.pl    | 1 +
>  langs/en/cgi-bin/en.pl    | 1 +
>  3 files changed, 10 insertions(+)
>  mode change 100644 => 100755 html/cgi-bin/netother.cgi
> 
> diff --git a/html/cgi-bin/netother.cgi b/html/cgi-bin/netother.cgi
> old mode 100644
> new mode 100755
> index dde1b603a..4f03c82e8
> --- a/html/cgi-bin/netother.cgi
> +++ b/html/cgi-bin/netother.cgi
> @@ -79,6 +79,14 @@ if ( $querry[0] =~ "fwhits"){
>  	print "<pre>$output</pre>\n";
>  	&Header::closebox();
> 
> +	$output = `/sbin/ip route list table 220`;
> +	if ( $output ) {
> +		&Header::openbox('100%', 'left', $Lang::tr{'ipsec routing table 
> entries'});
> +		$output = &Header::cleanhtml($output,"y");
> +		print "<pre>$output</pre>\n";
> +		&Header::closebox()
> +	}
> +
>  	&Header::openbox('100%', 'left', $Lang::tr{'arp table entries'});
>  	$output = `/sbin/ip neigh show`;
>  	$output = &Header::cleanhtml($output,"y");
> diff --git a/langs/de/cgi-bin/de.pl b/langs/de/cgi-bin/de.pl
> index 80579e7cc..6f7b4db59 100644
> --- a/langs/de/cgi-bin/de.pl
> +++ b/langs/de/cgi-bin/de.pl
> @@ -1498,6 +1498,7 @@
>  'ipsec mode tunnel' => 'Tunnel',
>  'ipsec network' => 'IPsec-Netzwerk',
>  'ipsec no connections' => 'Keine aktiven IPsec-Verbindungen',
> +'ipsec routing table entries' => 'Einträge der IPsec-Routing-Tabelle',
>  'ipsec settings' => 'IPsec-Einstellungen',
>  'iptable rules' => 'IPTable-Regeln',
>  'iptmangles' => 'IPTable Mangles',
> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
> index a68c8f411..1e1aed53c 100644
> --- a/langs/en/cgi-bin/en.pl
> +++ b/langs/en/cgi-bin/en.pl
> @@ -1547,6 +1547,7 @@
>  'ipsec mode tunnel' => 'Tunnel',
>  'ipsec network' => 'IPsec network',
>  'ipsec no connections' => 'No active IPsec connections',
> +'ipsec routing table entries' => 'IPsec Routing Table Entries',
>  'ipsec settings' => 'IPsec Settings',
>  'iptable rules' => 'IPTable rules',
>  'iptmangles' => 'IPTable Mangles',
  
Peter Müller March 30, 2020, 5:56 p.m. UTC | #3
Hello *,

for your information: The patch was already applied by Arne at March 26th, but the
Patchwork did not get it and so Git complained when he tried to apply it again.

Everything is fine, move along. :-)

Thanks, and best regards,
Peter Müller


> This patch doesn't apply. It also change the rights of the netother.cgi in git. This should be 644 not 755
> 
> 
> Am 2020-03-21 17:03, schrieb Peter Müller:
>> Since IPsec routing information do not show up in the normal routing
>> table, also displaying the contents of table 220 on netother.cgi might
>> be useful for debugging purposes.
>>
>> The second version of this patch omits the output if routing table 220
>> is empty and introduces a custom translation for IPsec routing table
>> entries instead of just adding the table number to the generic translation.
>>
>> Cc: Michael Tremer <michael.tremer@ipfire.org>
>> Signed-off-by: Peter Müller <peter.mueller@ipfire.org>
>> ---
>>  html/cgi-bin/netother.cgi | 8 ++++++++
>>  langs/de/cgi-bin/de.pl    | 1 +
>>  langs/en/cgi-bin/en.pl    | 1 +
>>  3 files changed, 10 insertions(+)
>>  mode change 100644 => 100755 html/cgi-bin/netother.cgi
>>
>> diff --git a/html/cgi-bin/netother.cgi b/html/cgi-bin/netother.cgi
>> old mode 100644
>> new mode 100755
>> index dde1b603a..4f03c82e8
>> --- a/html/cgi-bin/netother.cgi
>> +++ b/html/cgi-bin/netother.cgi
>> @@ -79,6 +79,14 @@ if ( $querry[0] =~ "fwhits"){
>>      print "<pre>$output</pre>\n";
>>      &Header::closebox();
>>
>> +    $output = `/sbin/ip route list table 220`;
>> +    if ( $output ) {
>> +        &Header::openbox('100%', 'left', $Lang::tr{'ipsec routing table entries'});
>> +        $output = &Header::cleanhtml($output,"y");
>> +        print "<pre>$output</pre>\n";
>> +        &Header::closebox()
>> +    }
>> +
>>      &Header::openbox('100%', 'left', $Lang::tr{'arp table entries'});
>>      $output = `/sbin/ip neigh show`;
>>      $output = &Header::cleanhtml($output,"y");
>> diff --git a/langs/de/cgi-bin/de.pl b/langs/de/cgi-bin/de.pl
>> index 80579e7cc..6f7b4db59 100644
>> --- a/langs/de/cgi-bin/de.pl
>> +++ b/langs/de/cgi-bin/de.pl
>> @@ -1498,6 +1498,7 @@
>>  'ipsec mode tunnel' => 'Tunnel',
>>  'ipsec network' => 'IPsec-Netzwerk',
>>  'ipsec no connections' => 'Keine aktiven IPsec-Verbindungen',
>> +'ipsec routing table entries' => 'Einträge der IPsec-Routing-Tabelle',
>>  'ipsec settings' => 'IPsec-Einstellungen',
>>  'iptable rules' => 'IPTable-Regeln',
>>  'iptmangles' => 'IPTable Mangles',
>> diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
>> index a68c8f411..1e1aed53c 100644
>> --- a/langs/en/cgi-bin/en.pl
>> +++ b/langs/en/cgi-bin/en.pl
>> @@ -1547,6 +1547,7 @@
>>  'ipsec mode tunnel' => 'Tunnel',
>>  'ipsec network' => 'IPsec network',
>>  'ipsec no connections' => 'No active IPsec connections',
>> +'ipsec routing table entries' => 'IPsec Routing Table Entries',
>>  'ipsec settings' => 'IPsec Settings',
>>  'iptable rules' => 'IPTable rules',
>>  'iptmangles' => 'IPTable Mangles',
  

Patch

diff --git a/html/cgi-bin/netother.cgi b/html/cgi-bin/netother.cgi
old mode 100644
new mode 100755
index dde1b603a..4f03c82e8
--- a/html/cgi-bin/netother.cgi
+++ b/html/cgi-bin/netother.cgi
@@ -79,6 +79,14 @@  if ( $querry[0] =~ "fwhits"){
 	print "<pre>$output</pre>\n";
 	&Header::closebox();
 
+	$output = `/sbin/ip route list table 220`;
+	if ( $output ) {
+		&Header::openbox('100%', 'left', $Lang::tr{'ipsec routing table entries'});
+		$output = &Header::cleanhtml($output,"y");
+		print "<pre>$output</pre>\n";
+		&Header::closebox()
+	}
+
 	&Header::openbox('100%', 'left', $Lang::tr{'arp table entries'});
 	$output = `/sbin/ip neigh show`;
 	$output = &Header::cleanhtml($output,"y");
diff --git a/langs/de/cgi-bin/de.pl b/langs/de/cgi-bin/de.pl
index 80579e7cc..6f7b4db59 100644
--- a/langs/de/cgi-bin/de.pl
+++ b/langs/de/cgi-bin/de.pl
@@ -1498,6 +1498,7 @@ 
 'ipsec mode tunnel' => 'Tunnel',
 'ipsec network' => 'IPsec-Netzwerk',
 'ipsec no connections' => 'Keine aktiven IPsec-Verbindungen',
+'ipsec routing table entries' => 'Einträge der IPsec-Routing-Tabelle',
 'ipsec settings' => 'IPsec-Einstellungen',
 'iptable rules' => 'IPTable-Regeln',
 'iptmangles' => 'IPTable Mangles',
diff --git a/langs/en/cgi-bin/en.pl b/langs/en/cgi-bin/en.pl
index a68c8f411..1e1aed53c 100644
--- a/langs/en/cgi-bin/en.pl
+++ b/langs/en/cgi-bin/en.pl
@@ -1547,6 +1547,7 @@ 
 'ipsec mode tunnel' => 'Tunnel',
 'ipsec network' => 'IPsec network',
 'ipsec no connections' => 'No active IPsec connections',
+'ipsec routing table entries' => 'IPsec Routing Table Entries',
 'ipsec settings' => 'IPsec Settings',
 'iptable rules' => 'IPTable rules',
 'iptmangles' => 'IPTable Mangles',