Hello,

Saturday, July 31, 2004, 7:29:20 PM, Mark Powell wrote:
> Patch against current CVS (as all the others have been, even if I forgot
> to mention it). This is a further development on the last patch I posted a
> couple of months ago. This does rfc checking, but also proper background
> DNS checking of the helo argument if so configured. The docs should be up
> to date.

patched version:
sub register {
  my ($self, $qp, @args) = @_;

  return if exists $ENV{RELAYCLIENT};

previous version: (the one I wrote a diff to work properly under pperl)
sub check_helo {
  my ($self, $transaction, $hello) = @_;
  my $remote_ip = $self->qp->connection->remote_ip;

  return (DECLINED) if !$self->{_spamhelo_checkrc} && exists $ENV{RELAYCLIENT};

shouldn't the RELAYCLIENT test be somewhere in sub check_helo as in previous
version? Maybe this is only an issue under pperl (again ;)

-- 
spaze -- exploited.cz -- tojeono.cz
- forget everything and remember! -

Reply via email to