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