[Mimedefang] SURBL lookups no longer happening after upgrade to2.48

[Sven Willenberger]

On Fri, 2004-12-10 at 15:10 -0600, Mike Carlson wrote:
> I am having the same problem with SA 3.01 and MIMEDefang 2.48 on FreeBSD. I finally did get it working.
>  
> I noticed in /usr/loca/bin/mimedefang.pl that LOCAL_RULES_DIR was defined as:
>  
> hades# grep LOCAL_RULES /usr/local/bin/mimedefang.pl
>     my $LOCAL_RULES_DIR = "/usr/local/etc/mimedefang/spamassassin";
>             LOCAL_RULES_DIR    => $LOCAL_RULES_DIR});
>            # LOCAL_RULES_DIR    => $LOCAL_RULES_DIR,
>  
> So I copied init.pre from /usr/local/etc/mail/spamassassin/ to /usr/local/etc/mimedefang/spamassassin/
>  
> Now SURBLS are hitting:
>  
> Dec 10 15:02:19 hades sm-mta[346]: iBAL2DTM000346: Milter change: header  X-Spam-Score: from 9.258 to 9.258 (*********) BAD_CREDIT,DCC_CHECK,DIGEST_MULTIPLE,EXCUSE_7,HTML_50_60,HTML_FONT_SIZE_TINY,HTML_FONT_TINY,HTML_IMAGE_ONLY_24,HTML_MESSAGE,HTML_TEXT_AFTER_BODY,HTML_TEXT_AFTER_HTML,MIME_BOUND_NEXTPART,MSGID_FROM_MTA_HEADER,RAZOR2_CF_RANGE_51_100,RAZOR2_CHECK,URIBL_OB_SURBL,URIBL_SBL,URIBL_WS_SURBL
>  
> I was not getting any hits earlier today. Since this seemed to fix it I think I may just take the init.pre in /usr/local/etc/mimedefang/spamassasin/ and make it a symbolic link to the one in /usr/local/etc/mail/spamassassin/ directory. That way I only have to manage one file.
>  
> --Mike
>  
> 
> ________________________________
> 
> From: mimedefang-bounces at lists.roaringpenguin.com on behalf of Rob MacGregor
> Sent: Fri 12/10/2004 1:47 PM
> To: mimedefang at lists.roaringpenguin.com
> Subject: Re: [Mimedefang] SURBL lookups no longer happening after upgrade to2.48
> 
> 
> 
> Ok, putting the test into local.cf got me the following error:
> 
> ... mimedefang-multiplexor[50777]: Slave 0 stderr: Failed to run
> URIBL_SC_SURBL SpamAssassin test, skipping:        (Can't locate
> object method "check_uridnsbl" via package
> "Mail::SpamAssassin::PerMsgStatus" at
> /usr/local/lib/perl5/site_perl/5.8.5/Mail/SpamAssassin/PerMsgStatus.pm
> line 2296. )
> 
> Which is what happens if the module isn't loaded, so it looks like
> something isn't happening with init.pre when called from MD.
> 
> Sticking the loadplugin command into local.cf AND importing all the
> UIRBL lines makes it work, but there's obviously something broken :(
> 

This behavior could explain why the order of $config and LOCAL_RULES_DIR
seems to make a difference in mimedefang.pl. In the default
installation, LOCAL_RULES_DIR is set in the hash when passed to the new
constructor, followed by $config (which reads sa-mimedefang.cf). At that
point, SA sees the empty LOCAL_RULES_DIR and doesn't know about
init.pre. By putting LOCAL_RULES_DIR after $config in the hash, SA sees
$config and in order to parse $config loads the init.pre from the
default location, then it sees the LOCAL_RULES_DIR. Although hash order
should not make a difference, if spamassassin loops through the hash
(for $keys .....), it may parse them in order causing this behavior.

Sven