SH_BODYURI_REVERSE_SBL uses check_sh_bodyuri_a() uses non-working uridnsbl_skip_domains

[robert-scheck]

Moving #68 (comment) as kindly requested by @xrat in #68 (comment) into an own issue:

spamassassin-dqs/3.4.1+/SH.pm

Line 707 in 666784c

if (!($skip_domains->{$this_hostname})) {

does not work, @ricalfieri. SH_BODYURI_REVERSE_SBL uses check_sh_bodyuri_a() which aims to use a skip list via

spamassassin-dqs/3.4.1+/SH.pm

Line 697 in 666784c

my $skip_domains = $conf->{uridnsbl_skip_domains};

which suggests to be filled using uridnsbl_skip_domains, which however unfortunately has absolutely no effect (it's actually uridnsbl_skip_domain googleapis.com fonts.googleapis.com in my SpamAssassin configuration). My expectation is quite simply: Please really skip the domain as the SH.pm code makes admins believe.

[ricalfieri]

Hello, i'm not going to fix something on 3.4.x. If anyone wants to make a pull request with a fix it would be appreciated