Hi !!
> IIRC the '$1' in isip{} won't contain the IP address extracted by the
> match if you do it that way; it'll always end up empty. I put the
> isip{$1} inside the result of the ${if} on purpose.
in fact it could be more simple, as if the helo is enclosed in []
then it must be an ip literal, so it must match the ip address:
# Check ip literals
accept condition = ${if match {$sender_helo_name}\
{\N^\[(.*)\]$\N}{yes}{no}}
condition = ${if eq{$sender_helo_name}\
{[$sender_host_address]}{yes}{no}}
deny condition = ${if match {$sender_helo_name}\
{\N^\[(.*)\]$\N}{yes}{no}}
message = Forged HELO: you are not $sender_helo_name
log_message = Forged HELO: ip does not match
--
Best regards ...
----------------------------------------------------------------
David Saez Padros http://www.ols.es
On-Line Services 2000 S.L. e-mail david@???
Pintor Vayreda 1 telf +34 902 50 29 75
08184 Palau-Solita i Plegamans movil +34 670 35 27 53
----------------------------------------------------------------