[Logcheck-devel] Bug#445473: Updated patch for policyd-weight ignores

[Justin Pryzby]

On Sun, Mar 16, 2008 at 10:40:01PM +0200, Sami Haahtinen wrote:
> On Sun, 2008-03-16 at 22:36 +0200, Sami Haahtinen wrote:
> > On Sun, 2008-03-16 at 12:58 -0400, Frédéric Brière wrote:
> > > Your log didn't include an example for "temporarily blocked" and
> > > "multirecipient email", but those were clear enough anyway.  I also
> > > added "please relay via your ISP", which was not covered by your patch.
> > > 
> > > I was intrigued by your making optional of one space and all of
> > > client/helo/from/to in the "weighted check" rule.  Was that intentional?
> > 
> > I'm pretty sure there was a reason for that, but it could be that it was
> > a leftover from some variation of the rule. You can leave it out and
> > i'll file a new bug if it's required.
> 
> Hmm.. after re-reading that all, i was of course referring to the first
> optional space.
> 
> The client/helo/... is optional because it's not always included in the
> "decided action=PREPEND" form. I faintly remember considering using 2
> sets of rules for those lines but decided against it for some reason.
Are these optional due to use of delay_reject=no ?