Re: spam-check-spamassassin-headers with spamassasin 3.0/3.1

[IRIE Tetsuya <irie@t.email.ne.jp>]

[-- Attachment #1: Type: text/plain, Size: 673 bytes --]

Thank you for your reply.

Reiner Steib <reinersteib+gmane@imap.cc> writes:
> Could you try the following patch, please?

`spam-check-spamassassin-headers' works fine, however
`spam-extra-header-to-number' doesn't.

spamassasin add header like this:
| X-Spam-Status: Yes, score=13.1 required=5.0 tests=DNS_FROM_RFC_ABUSE,
| 	DNS_FROM_RFC_WHOIS,FORGED_YAHOO_RCVD,MISSING_SUBJECT,NO_REAL_NAME,
| 	UNDISC_RECIPS autolearn=disabled version=3.0.3

Score always becomes 0 at (string-to-number (gnus-replace-in-string
... )), because "Yes" or "No" flag exist before score=xxx.

When I changed spam.el as attached, the displayed score is corrected.

Best regards,
-- 
IRIE Tetsuya

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: spam.el.patch --]
[-- Type: text/x-patch, Size: 722 bytes --]

--- spam.el.orig	2006-02-26 10:46:08.533820872 +0900
+++ spam.el	2006-02-26 09:58:40.303817648 +0900
@@ -1184,10 +1184,11 @@
   (if (gnus-extra-header header headers)
       (cond
        ((eq header 'X-Spam-Status)
-	(string-to-number (gnus-replace-in-string
-			   (gnus-extra-header header headers)
-			   spam-spamassassin-score-regexp
-			   "\\1")))
+	(if (string-match spam-spamassassin-score-regexp
+			  (gnus-extra-header header headers))
+	    (string-to-number
+	     (match-string 1 (gnus-extra-header header headers)))
+	  0))
        ;; for CRM checking, it's probably faster to just do the string match
        ((and spam-use-crm114 (string-match "( pR: \\([0-9.-]+\\)" header))
 	(match-string 1 header))