You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by he...@apache.org on 2021/04/22 06:44:25 UTC

svn commit: r1889093 - /spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm

Author: hege
Date: Thu Apr 22 06:44:25 2021
New Revision: 1889093

URL: http://svn.apache.org/viewvc?rev=1889093&view=rev
Log:
Check full hostname from uridnsbl_skip_domain also

Modified:
    spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm

Modified: spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm
URL: http://svn.apache.org/viewvc/spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm?rev=1889093&r1=1889092&r2=1889093&view=diff
==============================================================================
--- spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm (original)
+++ spamassassin/trunk/lib/Mail/SpamAssassin/Plugin/URIDNSBL.pm Thu Apr 22 06:44:25 2021
@@ -57,6 +57,9 @@ Specify a domain, or a number of domains
 URIBL checks.  This is very useful to specify very common domains which are
 not going to be listed in URIBLs.
 
+In addition to trimmed domain, the full hostname is also checked from the
+list.
+
 =back
 
 =over 4
@@ -448,7 +451,11 @@ sub check_dnsbl {
     while (my($host,$domain) = each( %{$info->{hosts}} )) {
       if ($skip_domains->{$domain}) {
         dbg("uridnsbl: domain $domain in skip list, host $host");
-      } else {
+      }
+      elsif ($skip_domains->{$host}) {
+        dbg("uridnsbl: host $host in skip list, domain $domain");
+      }
+      else {
         # use hostname as a key, and drag along the stripped domain name part
         $uri_ordered[$entry]->{$host} = $domain;
       }
@@ -1152,6 +1159,6 @@ sub has_tflags_domains_only { 1 }
 sub has_subtest_for_ranges { 1 }
 sub has_uridnsbl_for_a { 1 }  # uridnsbl rules recognize tflags 'a' and 'ns'
 sub has_uridnsbl_a_ns { 1 }  # has an actually working 'a' flag, unlike above :-(
-sub has_tflags_notrim { 1 }
+sub has_tflags_notrim { 1 }  # Bug 7835
 
 1;