On Monday 30 July 2018 11:18:19 Jeremy Harris via Exim-users wrote:
> On 07/30/2018 10:47 AM, Gary Stainburn via Exim-users wrote:
> >   drop    message   = Subject is know SPAM email, ignoring
> >           log_message = Message dropped because of know SPAM Subject
>
> - dubious grammar in those messages
> - I prefer to put the conditions first

Good point.  This is only a test setup, but for clarity I have tidied up my 
code.

>
> >           condition = ${if >{${strlen:$h_subject:}}{20}{yes}{no}}
> >       condition =
> > ${lookup{"$h_subject:"}lsearch{/etc/exim/lists/subject.droplist}{yes}{no}
> >}
>
> - using "drop" rather than "deny" leaves the sender having to assume
> there was an error.  A positive rejection makes it more likely they
> won't come back, and on false-positives will leave better log
> information on the sending system for debugging

Agreed, drop changed to deny.

>
> - you don't say where you placed this verb.  If it's not in a data-time
> ACL, the headers are not available

This is in acl_check_data along with my other anti-SPAM/Virus code

>
> --
> Cheers,
>   Jeremy



-- 
Gary Stainburn
Group I.T. Manager
Ringways Garages
http://www.ringways.co.uk 

-- 
## List details at https://lists.exim.org/mailman/listinfo/exim-users
## Exim details at http://www.exim.org/
## Please use the Wiki with this list - http://wiki.exim.org/

Reply via email to