Re: Feature Request: postscreen support in collate.pl

2022-05-17 Thread Wietse Venema
Peter Ajamian: > Can we get postscreen support in collate.pl? In other words have it > group postscreen log entries with the rest of the log entries for a > connection, or just show postscreen log entries when postscreen defers > the connection? Postscreen should not be configu

Feature Request: postscreen support in collate.pl

2022-05-17 Thread Peter Ajamian
Can we get postscreen support in collate.pl? In other words have it group postscreen log entries with the rest of the log entries for a connection, or just show postscreen log entries when postscreen defers the connection? Peter Ajamian

Re: collate.pl

2016-08-29 Thread Viktor Dukhovni
On Mon, Aug 29, 2016 at 09:36:44AM -0400, Wietse Venema wrote: > Robert Dahlem: > > 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 entri

Re: collate.pl

2016-08-29 Thread Robert Dahlem
Hi, On 29.08.2016 16:41, Robert Dahlem wrote: > 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=$sysl

Re: collate.pl

2016-08-29 Thread Wietse Venema
/dev/rob0: > On Mon, Aug 29, 2016 at 09:36:44AM -0400, Wietse Venema wrote: > > Robert Dahlem: > > > 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 maste

Re: collate.pl

2016-08-29 Thread Robert Dahlem
s: 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/qm

Re: collate.pl

2016-08-29 Thread /dev/rob0
On Mon, Aug 29, 2016 at 09:36:44AM -0400, Wietse Venema wrote: > Robert Dahlem: > > 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 entr

Re: collate.pl

2016-08-29 Thread Wietse Venema
Robert Dahlem: > 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 - -

Re: collate.pl

2016-08-29 Thread Robert Dahlem
6A25A: removed Because it does no harm to anything existing I propose that attached patch gets into the distribution. Kind regards, Robert --- collate.pl.orig 2016-08-23 17:35:19.0 +0200 +++ collate.pl 2016-08-29 11:42:51.0 +0200 @@ -10,7 +10,9 @@ \A # Absol

Re: collate.pl

2016-08-23 Thread Viktor Dukhovni
> On Aug 23, 2016, at 12:19 PM, Robert Dahlem wrote: > > my $instre = qr{(?x) >\A # Absolute line start >(?:\S+ \s+){3} # Timestamp, adjust for other time formats >\S+ \s+ # Hostname >(postfix(?:-\S+)?)/ # postfix

collate.pl

2016-08-23 Thread Robert Dahlem
Hi, some days ago Wietse mentioned auxiliary/collate/collate.pl in the Postfix 3.2 source-code distribution. I did try my luck with it and it is pretty impressive. There remains one problem for me: it got this regular expression: my $instre = qr{(?x) \A # Absolute