On Fri, 20 Aug 2004, spaze wrote:
> 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 ;)
Yeah. I fixed that locally since posting after, being chided by Ask for
putting it there :) I didn't submit a new patch. I've posted patches about
4 times for check_spamhelo, but they never make it into cvs, so... Which
is probably a good reason as they all seemed to have bugs in them :(
Cheers.
--
Mark Powell - UNIX System Administrator - The University of Salford
Information Services Division, Clifford Whitworth Building,
Salford University, Manchester, M5 4WT, UK.
Tel: +44 161 295 4837 Fax: +44 161 295 5888 www.pgp.com for PGP key
