Bug 1868 - X-Spam-Checker-Version header inconsistent in use of parens
Summary: X-Spam-Checker-Version header inconsistent in use of parens
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Libraries (show other bugs)
Version: 2.53
Hardware: All All
: P5 trivial
Target Milestone: 2.54
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords: backport
Depends on:
Blocks:
 
Reported: 2003-05-07 11:12 UTC by Sean Harding
Modified: 2003-05-07 08:54 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
suggested patch patch None Sean Harding [NoCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Harding 2003-05-07 11:12:57 UTC 
If a message is spam, X-Spam-Checker-Version is:

SpamAssassin 2.53 1.174.2.15-2003-03-30-exp

If it's non-spam, X-Spam-Checker-Version is:

SpamAssassin 2.53 (1.174.2.15-2003-03-30-exp)

Notice that the second case has parentheses while the first doesn't. The
offending code is in PerMsgStatus.pm at lines 630 and 712 (at least in 2.53).
Comment 1 Sean Harding 2003-05-07 11:20:06 UTC 
Created attachment 939 [details]
suggested patch
Comment 2 Theo Van Dinter 2003-05-07 15:06:59 UTC 
OKAY: looks good to me.
Comment 3 Justin Mason 2003-05-07 15:09:01 UTC 
OKAY: me too
Comment 4 Daniel Quinlan 2003-05-07 15:09:33 UTC 
OKAY: looks good to me for both trees
Comment 5 Theo Van Dinter 2003-05-07 15:24:11 UTC 
applied to stable and head.