Re: [PATCH pmg-gui v2 4/5] fix #3657: system options: add log-tracker-base textrow

["Max R. Carrara" <m.carrara@proxmox.com>]

On Tue Jun 9, 2026 at 10:04 PM CEST, Stoiko Ivanov wrote:
> the system options panel seems like the best place to set the location
> of the logs.
>
> Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
> ---
>  js/SystemOptions.js | 6 ++++++
>  1 file changed, 6 insertions(+)
>
> diff --git a/js/SystemOptions.js b/js/SystemOptions.js
> index 92dc5d0..b7d20fd 100644
> --- a/js/SystemOptions.js
> +++ b/js/SystemOptions.js
> @@ -91,6 +91,12 @@ Ext.define('PMG.SystemOptions', {
>              renderer: Ext.htmlEncode,
>          });
>
> +        me.add_text_row('log-tracker-base', gettext("Maillog base name"), {
> +            deleteEmpty: true,
> +            defaultValue: '/var/log/syslog',
> +            renderer: Ext.htmlEncode,
> +        });
> +

Even though it's kind of superfluous, perhaps the config option could be
greyed out or disabled in some way if the currently logged in user isn't
root@pam? Only a small thing though -- which probably depends on
personal taste as well -- so no hard feelings if this stays as it is.

>          me.add_proxy_row('http_proxy', gettext('HTTP proxy'));
>
>          me.add_textareafield_row('consent-text', gettext('Consent Text'), {