Re: [PATCH pmg-api v2] config: do not shortcircuit smtp-welcomelist generation

[Dominik Csapak <d.csapak@proxmox.com>]

after applying these changes and rebooting, receiving mail works again.
changes look sensible, so

Tested-by: Dominik Csapak <d.csapak@proxmox.com>
Reviewed-by: Dominik Csapak <d.csapak@proxmox.com>

On 6/9/26 11:04 AM, Stoiko Ivanov wrote:
> combining the results with ||= causes the write_smtp_welcomelist calls
> not to be run, after the first one signals a change.
> gather the combined changes in a different variable, to ensure each
> config-file is actually created/written, as postfix refuses to run if
> its config files are missing.
> 
> Fixes:  de0ba685 ("fix #3991: record changes to the smtp-welcomelist and reload postfix")
> Reported-by: Dominik Csapak <d.csapak@proxmox.com>
> Signed-off-by: Stoiko Ivanov <s.ivanov@proxmox.com>
> ---
> v1->v2:
> * fix the use of ||= I kept by mistake for the 3. and 4. call to
>    write_smtp_welcomlist - thanks to Dominik for spotting it!
> 
>   src/PMG/Config.pm | 13 +++++++++----
>   1 file changed, 9 insertions(+), 4 deletions(-)
> 
> diff --git a/src/PMG/Config.pm b/src/PMG/Config.pm
> index 2d2b6c83..f48c31dc 100644
> --- a/src/PMG/Config.pm
> +++ b/src/PMG/Config.pm
> @@ -1878,10 +1878,15 @@ sub rewrite_postfix_welcomelist {
>       }
>   
>       my $changes = 0;
> -    $changes ||= $write_smtp_welcomelist->("/etc/postfix/senderaccess", $fromlist);
> -    $changes ||= $write_smtp_welcomelist->("/etc/postfix/rcptaccess", $tolist);
> -    $changes ||= $write_smtp_welcomelist->("/etc/postfix/clientaccess", $clientlist);
> -    $changes ||= $write_smtp_welcomelist->("/etc/postfix/postscreen_access", $clientlist, 'permit');
> +    my $ret;
> +    $ret = $write_smtp_welcomelist->("/etc/postfix/senderaccess", $fromlist);
> +    $changes ||= $ret;
> +    $ret = $write_smtp_welcomelist->("/etc/postfix/rcptaccess", $tolist);
> +    $changes ||= $ret;
> +    $ret = $write_smtp_welcomelist->("/etc/postfix/clientaccess", $clientlist);
> +    $changes ||= $ret;
> +    $ret = $write_smtp_welcomelist->("/etc/postfix/postscreen_access", $clientlist, 'permit');
> +    $changes ||= $ret;
>   
>       return $changes;
>   }