You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@spamassassin.apache.org on 2023/03/15 17:14:46 UTC

[Bug 8121] header rules with keyword all not working

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=8121

Henrik Krohns <ap...@hege.li> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 OS|Linux                       |All
                 CC|                            |apache@hege.li
   Target Milestone|Undefined                   |4.0.1

--- Comment #1 from Henrik Krohns <ap...@hege.li> ---
Thanks, we are missing many tests for these..

Unfortunately the fix is not that simple. Removing "\n" from Check.pm breaks
other stuff like Cc:addr. The real culprit is pms->_get() returning
inconsistent line endings, this needs careful vetting and adding tests for all
cases.

In the meantime, I've committed the workaround for some tests:

Sending        rules/20_head_tests.cf
Sending        rulesrc/sandbox/jhardin/20_misc_testing.cf
Sending        rulesrc/sandbox/jm/20_basic.cf
Sending        rulesrc/sandbox/khopesh/20_khop_experimental.cf
Sending        rulesrc/sandbox/pds/10_menaces.cf
Transmitting file data .....done
Committing transaction...
Committed revision 1908419.

-- 
You are receiving this mail because:
You are the assignee for the bug.