> For aforementioned reasons, I'd like this to be configurable. I agree
> that DENYSOFT may be a better default.

Ask Ask said, make a patch.

It would be trivial to write a wrapper around the clamd plugin that
changed DENYSOFT to DECLINED.  It shouldn't take more than 6 lines
using plugin inheritance/wrapping.

Something like this:

# plugins/juerd_clamav
sub init {
  my ($self, $qp) = @_;
  $self->isa_plugin('rcpt_ok');
 }

sub hook_data_post {
  my ($self, $transaction, $recipient) = @_;
  my $r = $self->SUPER::hook_data_post( $transaction, $recipient );
  return $r == DENYSOFT ? DECLINED $r;
}


Ok.  10 lines,  but still.

I've played with methods of generalized mapping of return values from
hooks, but haven't come up with a clean enough way yet.

-R

Reply via email to