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

Reply via email to