You are viewing a plain text version of this content. The canonical link for it is here.
Posted to users@spamassassin.apache.org by Justin Mason <jm...@jmason.org> on 2007/08/02 18:54:32 UTC

Re: 3.2.1 RDNS_NONE bug/false positive

John, could you open this as a bug in bugzilla?

John Rudd writes:
> 
> 
> The following received header gets RDNS_NONE:
> 
> Received: from [69.12.154.165] (HELO mail.rudd.cc)
>    by some.internal.test.host (CommuniGate Pro SMTP 5.1.11)
>    with ESMTP id 60056 for jrudd@some.internal.test.domain; Tue, 31 Jul 
> 2007 14:05:04 -0700
> 
> 
> 
> This is an error in the RDNS_NONE logic.  The host in question DOES have 
> RDNS.  The issue is that CommuniGate Pro doesn't put the RDNS 
> information into the Received header (and the place where it puts the 
> HELO information might instead have the authenticated user, if SMTP-AUTH 
> was used).
> 
> 
> My recommendation is that the logic for RDNS_NONE should be:
> 
> 
> if (Received =~ /\(CommuniGate Pro SMTP /) {
>     if (DNS checks are enabled) {
>        do the DNS check on the IP address, and then decide if it has RDNS
>        }
>     else {
>        don't trigger RDNS_NONE (because you can't know without the DNS 
> check)
>        }
>     }
> else { # not a CommuniGate Pro MTA
>     do existing RDNS_NONE logic
>     }

Re: 3.2.1 RDNS_NONE bug/false positive

Posted by John Rudd <jr...@ucsc.edu>.
Done.  I also added a note wrt to Matthew Yette's comment that it 
applies to qmail-1.03 as well.

It's bug 5586.


Justin Mason wrote:
> John, could you open this as a bug in bugzilla?
> 
> John Rudd writes:
>>
>> The following received header gets RDNS_NONE:
>>
>> Received: from [69.12.154.165] (HELO mail.rudd.cc)
>>    by some.internal.test.host (CommuniGate Pro SMTP 5.1.11)
>>    with ESMTP id 60056 for jrudd@some.internal.test.domain; Tue, 31 Jul 
>> 2007 14:05:04 -0700
>>
>>
>>
>> This is an error in the RDNS_NONE logic.  The host in question DOES have 
>> RDNS.  The issue is that CommuniGate Pro doesn't put the RDNS 
>> information into the Received header (and the place where it puts the 
>> HELO information might instead have the authenticated user, if SMTP-AUTH 
>> was used).
>>
>>
>> My recommendation is that the logic for RDNS_NONE should be:
>>
>>
>> if (Received =~ /\(CommuniGate Pro SMTP /) {
>>     if (DNS checks are enabled) {
>>        do the DNS check on the IP address, and then decide if it has RDNS
>>        }
>>     else {
>>        don't trigger RDNS_NONE (because you can't know without the DNS 
>> check)
>>        }
>>     }
>> else { # not a CommuniGate Pro MTA
>>     do existing RDNS_NONE logic
>>     }