On 2/8/19 2:08 PM, Wietse Venema wrote:
Robert Moskowitz:

On 2/8/19 1:44 PM, Wietse Venema wrote:
Viktor Dukhovni:
pickup         unix   n             -             n             60           1  
           pickup
pickup         unix   n             -             n             60           1  
           pickup
          -o content_filter=
The "pickup" service is defined twice in master.c, the second
instance (last one wins) disables content filtering for mail submitted
locally via sendmail(1).
That was easy enough to fix:
When I was working on this 2 years ago, I thought it was kind of cool
that instead of editing master.cf entries to fix them, I could just
append a whole new entry with the 'right' content.

Much easier to automate changes (as we had nothing like postconf -e for
changing master.cf).? If I read the patch right, you are providing a
warning of the double entry.? Perhaps a better patch would warn and drop
all but the last entry?
Why do you think it was keeping both pickup entries?

Well, I am not sure.  From Viktor's earlier note, it seems that the last wins and the earlier ones are just ignored.  Maybe it is that Viktor said, "master.c" and I don't know what "master.c" different from "master.cf" that is in /etc/postfix.

I kind of assumed (and we know what that is an abbreviation for) that "master.c" is an internal entry in postfix built from processing master.cf.  Thus why keep all but the last in the internal table?


Reply via email to