Re: `S t' always returns nil

[Ted Zlatanov <tzz@lifelogs.com>]

On Sun, 26 Jan 2003, chris@splinter.inka.de wrote:
> Hi,
> 
> I think there is something wrong with `S t' as it always
> returns a spamicity of 0, even if the concerned mail is clearly
> spam and is recognized as such by bogofilter, too. 
> 
> I use the most recent version of bogofilter from CVS, which
> returns lines like this:
> 
>| X-Bogosity: Spam, tests=bogofilter, spamicity=0.9056373309,
>| version=0.10.1.1.cvs.20030125
> 
> The variables which are related to bogofilter are set properly.

Hmm, the version I used for testing had lines like this:

X-Bogosity: Yes, tests=bogofilter, spamicity=0.567040, version=0.9.1.2

so I wrote spam-check-bogofilter-headers to look for "X-Bogosity: Yes"
headers, see below.

(defun spam-check-bogofilter-headers (&optional score)
  (let ((header (message-fetch-field spam-bogofilter-header)))
      (when (and header
	       (string-match "^Yes" header))
	  (if score
	      (when (string-match "spamicity=\\([0-9.]+\\)" header)
		(match-string 1 header))
	    spam-split-group))))

Fixing this is trivial, but should I match on something else, or
should I allow both ^Yes and ^Spam as valid spam indicators in the
header?

Thanks
Ted