Bug 3846 - Verify RE before including a rule
Summary: Verify RE before including a rule
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Rules (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P5 enhancement
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-09-29 07:04 UTC by Theo Van Dinter
Modified: 2019-06-18 15:22 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 Theo Van Dinter 2004-09-29 07:04:42 UTC 
Per bug 3825, if a rule includes invalid RE, it will be included in the configuration, and then cause a 
whole set of rules to bomb when the RE isn't parsable by perl.

Before we accept a rule into the configuration, it should be verified to be valid, or at least mostly valid, 
depending on how paranoid we decide to be as a group. ;)
Comment 1 Bob Menschel 2005-04-09 14:50:57 UTC 
Per triage guidelines, picking a milestone. As a user who develops my own rules,
I'd prefer implementation by 3.1.1 if possible...
Comment 2 Daryl C. W. O'Shea 2005-05-10 22:22:09 UTC 
fixed in r169589
Comment 3 Daryl C. W. O'Shea 2005-05-11 00:06:13 UTC 
Not quite fixed.
Comment 4 Daniel Quinlan 2005-05-11 00:34:17 UTC 
this is fixed for real now
Comment 5 Justin Mason 2005-12-01 03:09:42 UTC 
reopening due to backout of fix, in bug 4700
Comment 6 Justin Mason 2005-12-01 03:14:14 UTC 
reassigning to dev
Comment 7 Henrik Krohns 2019-06-18 15:22:59 UTC 
It's properly fixed 14 years later. ;-)