Hi,

these should actually be two patches. One part is okay and I would merge that right away. For the second part I have a question :)

On Sat, 2016-05-21 at 14:31 +0200, Matthias Fischer wrote:

Clamav: Fix identation

Added entries in dropdown for 'Squid' and 'Snort'.

Signed-off-by: Matthias Fischer matthias.fischer@ipfire.org

html/cgi-bin/logs.cgi/log.dat | 10 +++++++---  1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/html/cgi-bin/logs.cgi/log.dat b/html/cgi-bin/logs.cgi/log.dat index e8635d7..de5076f 100644 --- a/html/cgi-bin/logs.cgi/log.dat +++ b/html/cgi-bin/logs.cgi/log.dat @@ -65,7 +65,9 @@ my %sections = (          'ipsec' => '(ipsec_[\w_]+: |pluto[.*]: |charon: |vpnwatch: )',          'openvpn' => '(openvpnserver[.*]: |.*n2n[.*]: )',          'pakfire' => '(pakfire:) ', -        'wireless' => '(hostapd:|kernel: ath.*:|kernel: wifi[0-9]:) ' +        'wireless' => '(hostapd:|kernel: ath.*:|kernel: wifi[0-9]:) ', +        'squid' => '(squid[.*]: |squid: )', +        'snort' => '(snort[.*]: )'    );

# Translations for the %sections array. @@ -78,7 +80,7 @@ my %trsections = (          'dhcp' => "$Lang::tr{'dhcp server'}",          'cron' => 'Cron',          'collectd' => 'Collectd',

• 		'clamav' => 'ClamAV',
  

+        'clamav' => 'ClamAV',          'ntp' => 'NTP',          'ssh' => 'SSH',          'auth' => "$Lang::tr{'loginlogout'}",

This looks fine. Should be its own patch.

@@ -86,7 +88,9 @@ my %trsections = (          'ipsec' => 'IPSec',          'openvpn' => 'OpenVPN',          'pakfire' => 'Pakfire', -        'wireless' => 'Wireless' +        'wireless' => 'Wireless', +        'squid' => 'Squid', +        'snort' => 'Snort'   );

Can't we name these "Intrusion Detection System" and "Web Proxy"?

This would make more sense to the lesser-experienced users. Are these also translated?

-Michael