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

[Bernhard Bitsch <bbitsch@ipfire.org>]

[-- 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\[.*\]: )',