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...@bugzilla.spamassassin.org on 2006/10/03 07:22:03 UTC

[Bug 5116] New: header RULENAME header:raw shouldn't strip leading spaces

http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116

           Summary: header RULENAME header:raw shouldn't strip leading
                    spaces
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: spamassassin@dostech.ca


It should be possible to write header rules to detect leading spaces in a header
value field.

:raw should include any leading and trailing spaces.  Without :raw, they should
probably continue to be striped.

This probably shouldn't be changed in 3.1 since it's a change in behaviour.



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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116





------- Additional Comments From spamassassin@dostech.ca  2006-10-13 11:32 -------
related bug 5127



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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P5                          |P2






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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116


spamassassin@dostech.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.2.0






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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116





------- Additional Comments From jm@jmason.org  2006-10-15 13:37 -------
yes -- I think it would make sense if :raw included those spaces.  agreed!



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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116





------- Additional Comments From jm@jmason.org  2006-11-09 14:15 -------
oh BTW -- in 3.1.x, it's probably easiest to use ALL.



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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116





------- Additional Comments From jm@jmason.org  2006-11-09 14:15 -------
Created an attachment (id=3744)
 --> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=3744&action=view)
implementation

here's the implementation; with this, ":raw" headers are
entirely pristine w.r.t. whitespace.  also adds a test or two.



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

[Bug 5116] header RULENAME header:raw shouldn't strip leading spaces

Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5116


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED




------- Additional Comments From jm@jmason.org  2006-11-11 07:30 -------
ok, checked in.  I don't think this should go to 3.1.x though

: jm 485...; svn commit -m "bug 5116: 'header RULENAME header:raw' shouldn't
strip spaces.  ensure it keeps a pristine copy of the header string for
matching, and use the test added in the previous checkin (oops ;)"
Sending        lib/Mail/SpamAssassin/Message/Node.pm
Sending        lib/Mail/SpamAssassin/Message.pm
Sending        lib/Mail/SpamAssassin/PerMsgStatus.pm
Transmitting file data ...

Committed revision 473730.




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