Bug 7382 - FP due to __BIGNUM_EMAILS
Summary: FP due to __BIGNUM_EMAILS
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Rules (show other bugs)
Version: 3.4.1
Hardware: PC Linux
: P2 normal
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-12-07 17:12 UTC by Kris Deugau
Modified: 2016-12-10 20:54 UTC (History)
1 user (show)


Attachment Type Modified Status Actions Submitter/CLA Status
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 Kris Deugau 2016-12-07 17:12:27 UTC 
I recently received a false-positive report where one of the key rules was BIGNUMS_EMAILS.

Checking the message showed the subrule __BIGNUMS_EMAILS triggering on the text "8,500 place names".

I suggest adding another exclusion for "place":

body __BIGNUM_EMAILS /\b(?:thousand|million|\d[,\d]{4,})\s(?:(?!and|or|your|place)\w+\s)?(?:e-?mail\saddresses|leads|names)\b/i
Comment 1 John Hardin 2016-12-10 20:54:59 UTC 
$ svn commit 20_misc_testing.cf 
Sending        20_misc_testing.cf
Transmitting file data .done
Committing transaction...
Committed revision 1773539.