[PATCH] log.dat: Fix an error with 'monit' logging too much data.

[PATCH] log.dat: Fix an error with 'monit' logging too much data.

[Matthias Fischer]

[-- Attachment #1: Type: text/plain, Size: 1047 bytes --]

Making the regex "non-greedy" fixes an error Jon found.

'monit' logged a whole line from '/var/log/messages' where it should only log the first part.

Reference:
https://www.ultraedit.com/support/tutorials-power-tips/ultraedit/non-greedy-perl-regex.html

Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
---
 html/cgi-bin/logs.cgi/log.dat | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/html/cgi-bin/logs.cgi/log.dat b/html/cgi-bin/logs.cgi/log.dat
index 0d6c6722a..3364b7bea 100644
--- a/html/cgi-bin/logs.cgi/log.dat
+++ b/html/cgi-bin/logs.cgi/log.dat
@@ -62,7 +62,7 @@ my %sections = (
         'ipfire' => '(ipfire: )',
         'ipsec' => '(ipsec_[\w_]+: |pluto\[.*\]: |charon: |vpnwatch: )',
         'kernel' => '(kernel: (?!DROP_))',
-        'monit' => '(monit\[.*\]: )',
+        'monit' => '(monit\[.*?\]: )',
         'ntp' => '(ntpd(?:ate)?\[.*\]: )',
 	'oinkmaster' => '(oinkmaster\[.*\]: )',
         'openvpn' => '(openvpnserver\[.*\]: |.*n2n\[.*\]: )',
-- 
2.25.1

Re: [PATCH] log.dat: Fix an error with 'monit' logging too much data.

[Peter Müller]

[-- Attachment #1: Type: text/plain, Size: 1141 bytes --]

Reviewed-by: Peter Müller <peter.mueller(a)ipfire.org>

> Making the regex "non-greedy" fixes an error Jon found.
> 
> 'monit' logged a whole line from '/var/log/messages' where it should only log the first part.
> 
> Reference:
> https://www.ultraedit.com/support/tutorials-power-tips/ultraedit/non-greedy-perl-regex.html
> 
> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
> ---
>  html/cgi-bin/logs.cgi/log.dat | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/html/cgi-bin/logs.cgi/log.dat b/html/cgi-bin/logs.cgi/log.dat
> index 0d6c6722a..3364b7bea 100644
> --- a/html/cgi-bin/logs.cgi/log.dat
> +++ b/html/cgi-bin/logs.cgi/log.dat
> @@ -62,7 +62,7 @@ my %sections = (
>          'ipfire' => '(ipfire: )',
>          'ipsec' => '(ipsec_[\w_]+: |pluto\[.*\]: |charon: |vpnwatch: )',
>          'kernel' => '(kernel: (?!DROP_))',
> -        'monit' => '(monit\[.*\]: )',
> +        'monit' => '(monit\[.*?\]: )',
>          'ntp' => '(ntpd(?:ate)?\[.*\]: )',
>  	'oinkmaster' => '(oinkmaster\[.*\]: )',
>          'openvpn' => '(openvpnserver\[.*\]: |.*n2n\[.*\]: )',

Re: [PATCH] log.dat: Fix an error with 'monit' logging too much data.

[Bernhard Bitsch]

[-- Attachment #1: Type: text/plain, Size: 1195 bytes --]

Reviewed-by: Bernhard Bitsch <bbitsch(a)ipfire.org>

Am 21.01.2022 um 18:07 schrieb Matthias Fischer:
> Making the regex "non-greedy" fixes an error Jon found.
> 
> 'monit' logged a whole line from '/var/log/messages' where it should only log the first part.
> 
> Reference:
> https://www.ultraedit.com/support/tutorials-power-tips/ultraedit/non-greedy-perl-regex.html
> 
> Signed-off-by: Matthias Fischer <matthias.fischer(a)ipfire.org>
> ---
>   html/cgi-bin/logs.cgi/log.dat | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/html/cgi-bin/logs.cgi/log.dat b/html/cgi-bin/logs.cgi/log.dat
> index 0d6c6722a..3364b7bea 100644
> --- a/html/cgi-bin/logs.cgi/log.dat
> +++ b/html/cgi-bin/logs.cgi/log.dat
> @@ -62,7 +62,7 @@ my %sections = (
>           'ipfire' => '(ipfire: )',
>           'ipsec' => '(ipsec_[\w_]+: |pluto\[.*\]: |charon: |vpnwatch: )',
>           'kernel' => '(kernel: (?!DROP_))',
> -        'monit' => '(monit\[.*\]: )',
> +        'monit' => '(monit\[.*?\]: )',
>           'ntp' => '(ntpd(?:ate)?\[.*\]: )',
>   	'oinkmaster' => '(oinkmaster\[.*\]: )',
>           'openvpn' => '(openvpnserver\[.*\]: |.*n2n\[.*\]: )',