Bug 7219

Summary: Incorrect use of __BODY_TEXT_LINE
Product: Spamassassin Reporter: RW <rwmaillists>
Component: RulesAssignee: SpamAssassin Developer Mailing List <dev>
Status: NEW ---    
Severity: minor CC: apache, mrl
Priority: P2    
Version: unspecified   
Target Milestone: Undefined   
Hardware: All   
OS: All   
Whiteboard:

Description RW 2015-07-12 14:05:25 UTC
__BODY_TEXT_LINE counts the number of lines (i.e. paragraphs) in the body counting the Subject as one. Where this is used in __BODY_URI_ONLY and __EMPTY_BODY, no account is taken of the possibility that the the subject is missing or empty. This allows __EMPTY_BODY to fire when there is one paragraph in the body and __BODY_URI_ONLY when there is a second paragraph.

I suggest adding:

 header __NONEMPTY_SUBJ Subject =~ /\S/

and using it to adjust the arithmetic in __EMPTY_BODY and __BODY_URI_ONLY.
Comment 1 Bill Cole 2017-12-11 21:40:36 UTC
*** Bug 7519 has been marked as a duplicate of this bug. ***
Comment 2 Henrik Krohns 2019-08-21 07:34:06 UTC
Could someone please fix these rules, now that we have a nosubject tflag too? Needs a feature if check for it..

Seems Bill also made some test rules, but they are still just that?

billcole/21_bug_7219.cf
jhardin/20_misc_testing.cf