This patch fixes RDNS_NONE test. If the sender's reverse lookup fails,
Received header line will contain the sender's IP address enclosed in square
brackets (e.g. `[a.b.c.d]').


Henrik K wrote
> Can you describe what this patch achieves?  I've been using amavisd-milter
> for years, I thought it works well already..
> 
> 
> On Fri, Jan 11, 2019 at 04:30:12AM +0100, Henry Miller wrote:
>> Index: lib/Mail/SpamAssassin/Message/Metadata/Received.pm
>> ===================================================================
>> --- lib/Mail/SpamAssassin/Message/Metadata/Received.pm       (revision 
>> 1850997)
>> +++ lib/Mail/SpamAssassin/Message/Metadata/Received.pm       (working copy)
>> @@ -619,6 +619,18 @@
>>        }
>>      }
>>  
>> +    elsif (/ \(amavisd-milter\)\;/) {
>> +      # Received: from localhost ([127.0.0.1] [127.0.0.1])
>> +      # by mail.local (amavisd-milter);
>> +      # # Thu, 10 Jan 2019 17:15:10 -0500 (EST)
>> +      if ( /^(\S+) \((\S+) \[(${IP_ADDRESS})\]\) by (\S+) / ) {
>> +    $mta_looked_up_dns = 1;
>> +    $helo = $1; $rdns = $2; $ip = $3; $by = $4;
>> +    if ($rdns =~ /^\[/) { $rdns = ''; }
>> +    goto enough;
>> +      }
>> +    }
>> +
>>      elsif (/\(Scalix SMTP Relay/) {
>>        # from DPLAPTOP ( 72.242.176.162) by mail.puryear-it.com (Scalix
>> SMTP Relay 10.0.1.3) via ESMTP; Fri, 23 Jun 2006 16:39:47 -0500 (CDT)
>>        if (/^(\S+) \( ?(${IP_ADDRESS})\) by (\S+)/) {





--
Sent from: 
http://spamassassin.1065346.n5.nabble.com/SpamAssassin-Dev-f69836.html

Reply via email to