> sub action_discard_bounce ($) {
>         my($message) = @_;
>         # don't quarantine if all recipients are @sewingwitch.com
>         my $non_sewingwitch =  grep !/[EMAIL PROTECTED]/i, @Recipients;
>         # check for only recipient being
>         # hostmaster or info (almost certain spam so don't quarantine)
>         if ( $non_sewingwitch &&
>              ( (scalar @Recipients != 1) ||
>                ($Recipients[0] !~
>  /^(info|hostmaster)[EMAIL PROTECTED]/) )) {

Above, you first check for $non_sewingwitch, which presumably means that
the mail isn't destined for sewingwitch, but then you _and_ this condition
with the following  check for info or hostmaster as the only recipient.
Did you mean || instead of && ?



_______________________________________________
Visit http://www.mimedefang.org and http://www.canit.ca
MIMEDefang mailing list
[email protected]
http://lists.roaringpenguin.com/mailman/listinfo/mimedefang

Reply via email to