On Wednesday 06 March 2002 01:40 am, Matt Sergeant wrote: > What I suggest is that the body stripping code adds the subject header in. > > I'll apply this patch if there are no objections: > > diff -u -r1.79 PerMsgStatus.pm > --- lib/Mail/SpamAssassin/PerMsgStatus.pm 5 Mar 2002 17:44:51 -0000 > 1.79 > +++ lib/Mail/SpamAssassin/PerMsgStatus.pm 6 Mar 2002 09:40:12 -0000 > @@ -658,7 +658,7 @@ > > my $bodytext = $self->get_decoded_body_text_array(); > > - my $text = ''; > + my $text = "Subject: " . $self->get('subject', '') . "\n\n"; > my $lastwasmime = 0; > foreach $_ (@{$bodytext}) { > /^SPAM: / and next; # SpamAssassin markup
Here's a patch so that the LINES_OF_YELLING eval subroutine ignores the first line of the body, which is really the subject; that way it won't get triggered by a yelling subject. Index: lib/Mail/SpamAssassin/EvalTests.pm =================================================================== RCS file: /cvsroot/spamassassin/spamassassin/lib/Mail/SpamAssassin/EvalTests.pm,v retrieving revision 1.109 diff -u -3 -p -r1.109 EvalTests.pm --- lib/Mail/SpamAssassin/EvalTests.pm 4 Mar 2002 17:43:49 -0000 1.109 +++ lib/Mail/SpamAssassin/EvalTests.pm 6 Mar 2002 10:39:34 -0000 @@ -887,6 +996,10 @@ sub check_for_yelling { # Make local copy of body. my @lines = @{$body}; + + # Get rid of subject line, so we won't trigger on a subject + # which is yelling + shift (@lines); # Get rid of everything but upper AND lower case letters map (s/[^A-Za-z]//sg, @lines); -- Visit http://dmoz.org, the world's | Give a man a match, and he'll be warm largest human edited web directory. | for a minute, but set him on fire, and | he'll be warm for the rest of his life. [EMAIL PROTECTED] ICQ: 132152059 | _______________________________________________ Spamassassin-talk mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/spamassassin-talk