|Summary:||Incorrect use of __BODY_TEXT_LINE|
|Component:||Rules||Assignee:||SpamAssassin Developer Mailing List <dev>|
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