Hi,

On 29.08.2016 15:36, Wietse Venema wrote:

>> Because it does no harm to anything existing I propose that attached
>> patch gets into the distribution.

> It works with multi-instance names. What is the result for the stock
> master.cf file, which has entries as shown below?
> 
>     submission inet n       -       n       -       -       smtpd
>        -o syslog_name=postfix/submission
>        ...
>     smtps     inet  n       -       n       -       -       smtpd
>        -o syslog_name=postfix/smtps
>        ...

> This logs process names as: postfix/submission/smtpd, postfix/smtps/smtpd.
> 
> Sorry, I don't have time to parse the script.

This makes collate.pl with the proposed patch put
        postfix/submission/smtpd
        postfix/smtps/smtpd
in the same block as for example
        postfix/cleanup
        postfix/qmgr
        postfix/smtp
which is exactly the intended behavior.

There is a potential problem with multiple instances: $syslog_name
defaults to $multi_instance_name. You need to pay attention to change
this in /etc/postfix-sec/master.cf too.

Unfortunately, /dev/rob0's proposol to use
"syslog_name=$syslog_name/submission" results in
        warning: unreasonable macro call nesting: "syslog_name"

Kind regards,
Robert


Reply via email to