On 3/4/2014 9:18 AM, Kevin A. McGrail wrote:
On 3/4/2014 9:09 AM, Mark Martinec wrote:
Dieter Braun wrote:

# sa-update
rules: failed to run T_HEADER_FROM_DIFFERENT_DOMAINS test, skipping:
 (Can't locate object method "check_equal_from_domains" via package
   "Mail::SpamAssassin::PerMsgStatus" at (eval 1008) line 97.

You are missing a line:
  loadplugin Mail::SpamAssassin::Plugin::HeaderEval
in one of your .pre files.

That line is normally in v320.pre. You may have commented it out,
or the file v320.pre is missing from your SpamAssassin configuration
directory.

Btw, the rule T_HEADER_FROM_DIFFERENT_DOMAINS should have
been conditionalized to only apply if the plugin HeaderEval
is available.

Good call.  Working on that now.

Change committed, good catch Mark. I was moving in that direction but it's very clear from reviewing now.

Dieter, if you uncomment/add loadplugin Mail::SpamAssassin::Plugin::HeaderEval to your v320.pre for now, you should be good to go.

svn commit -m 'Adding HeaderEval condition to FREEMAIL_FORGED_FROMDOMAIN & HEADER_FROM_DIFFERENT_DOMAINS'
Sending        rulesrc/sandbox/kmcgrail/20_freemail.cf
Transmitting file data .
Committed revision 1574117.

Expect the rule to be fixed in a few days and you can revert.

regards,
KAM

Reply via email to