> I believe there is a bug in the 'spamassassin' plugin. It seems to leave
> older 'X-Spam-Status' Header lines added by another System intact, and add
> his own 'X-Spam-Status' Line, leaving the mail with two different
> X-Spam-Status.

This should fix it:

--- spamassassin.orig   Wed Sep 22 23:08:03 2004
+++ spamassassin        Wed Sep 22 23:08:36 2004
@@ -144,7 +144,7 @@
   my $line0 = <SPAMD>; # get the first protocol lines out
   if ($line0) {
     $self->log(6, "check_spam: spamd: $line0");
-    $transaction->header->add("X-Spam-Check-By", $self->qp->config('me'), 0);
+    $transaction->header->replace("X-Spam-Check-By", $self->qp->config('me'), 0);
   }

   my ($flag, $hits, $required);
@@ -162,8 +162,8 @@
   $flag = $flag eq 'True' ? 'Yes' : 'No';
   $self->log(6, "check_spam: finished reading from spamd");

-  $transaction->header->add('X-Spam-Flag', 'YES', 0) if ($flag eq 'Yes');
-  $transaction->header->add('X-Spam-Status',
+  $transaction->header->replace('X-Spam-Flag', 'YES', 0) if ($flag eq 'Yes');
+  $transaction->header->replace('X-Spam-Status',
                            "$flag, hits=$hits required=$required\n" .
                            "\ttests=$tests", 0);
   $self->log(5, "check_spam: $flag, hits=$hits, required=$required, " .





-kju

-- 
      It's an insane world, but i'm proud to be a part of it. -- Bill Hicks

Reply via email to