Re: [PATCH] log.dat: Fixes bug#12950 - adding apcupsd to system logs list

[Michael Tremer <michael.tremer@ipfire.org>]

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

Reviewed-by: Michael Tremer <michael.tremer(a)ipfire.org>

> On 21 Feb 2023, at 13:55, Adolf Belka <adolf.belka(a)ipfire.org> wrote:
> 
> - Patch tested out on my production system that has apcupsd running on it. APCUPS was
>   in the list of options in the system logs and entries from apcupsd were extracted
>   correctly in the wui.
> 
> Fixes: Bug#12950
> Tested-by: Adolf Belka <adolf.belka(a)ipfire.org>
> Signed-off-by: Adolf Belka <adolf.belka(a)ipfire.org>
> ---
> html/cgi-bin/logs.cgi/log.dat | 2 ++
> 1 file changed, 2 insertions(+)
> 
> diff --git a/html/cgi-bin/logs.cgi/log.dat b/html/cgi-bin/logs.cgi/log.dat
> index 5fb1fcd5d..bf71b3c9f 100644
> --- a/html/cgi-bin/logs.cgi/log.dat
> +++ b/html/cgi-bin/logs.cgi/log.dat
> @@ -49,6 +49,7 @@ $cgiparams{'ACTION'} = '';
> $cgiparams{'SECTION'} = 'ipfire';
> 
> my %sections = (
> +        'apcupsd' => '(apcupsd\[.*?\]: )',
>         'auth' => '(\w+\(pam_unix\)\[.*\]: )',
>         'wio' => '(wio:|wio\[.*\])',
>         'captive' => '(Captive:)',
> @@ -81,6 +82,7 @@ my %sections = (
> 
> # Translations for the %sections array.
> my %trsections = (
> +        'apcupsd' => 'APCUPS',
>         'auth' => "$Lang::tr{'loginlogout'}",
>         'wio' => 'Who Is Online?',
>         'captive' => $Lang::tr{'Captive'},
> -- 
> 2.39.2
>