You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by fe...@apache.org on 2005/02/24 19:36:54 UTC
svn commit: r155223 - spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm
Author: felicity
Date: Thu Feb 24 10:36:53 2005
New Revision: 155223
URL: http://svn.apache.org/viewcvs?view=rev&rev=155223
Log:
bug 4154: check for dns_available wasn't trying multiple times as it was supposed to. let a 'softfail' (ie: Net::DNS didn't die) do a retry.
Modified:
spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm
Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm
URL: http://svn.apache.org/viewcvs/spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm?view=diff&r1=155222&r2=155223
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Dns.pm Thu Feb 24 10:36:53 2005
@@ -640,11 +640,15 @@
my $domain = splice(@domains, rand(@domains), 1);
dbg("dns: trying ($retry) $domain...");
my $result = $self->lookup_ns($domain);
- if(defined $result && scalar @$result > 0) {
- if ( $result ) {
+ if(defined $result) {
+ if (scalar @$result > 0) {
dbg("dns: NS lookup of $domain succeeded => DNS available (set dns_available to override)");
$IS_DNS_AVAILABLE = 1;
last;
+ }
+ else {
+ dbg("dns: NS lookup of $domain failed, no results found");
+ next;
}
}
else {