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/09/07 14:29:48 UTC

[Bug 5089] New: Unable to add header with single digit zero

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

           Summary: Unable to add header with single digit zero
           Product: Spamassassin
           Version: 3.1.2
          Platform: PC
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P5
         Component: spamassassin
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: obriend@axonsolutions.com


In order to use MS Exchange SP2's "Intelligent" Message Filtering, I added 
this to my local.cf:

add_header ham SCL 0
add_header spam SCL 9

This results in the header "X-Spam-SCL: 9" to my spam messages, but my ham 
gets "X-Spam-SCL: ".  Is there a way to add the numeral zero to a header? 
These do *not* work:

add_header ham SCL \0
add_header ham SCL "0"

Per Theo Van Dinter:
In PerMsgStatus's rewrite functions, look for things like:

    my $line = $self->_process_header($header,$data) || "";

As far as I can tell, _process_header() also returns something defined, which
means the only time the '|| ""' bit gets used is when _process_header()
returns 0, like when a user requests it. ;)    Oops.  (first time in how many
years someone wants to add a header with a single 0 in it. <G>)



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

[Bug 5089] Unable to add header with single digit zero

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





------- Additional Comments From spamassassin@dostech.ca  2006-09-18 05:57 -------
Created an attachment (id=3696)
 --> (http://issues.apache.org/SpamAssassin/attachment.cgi?id=3696&action=view)
patch




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

[Bug 5089] [review] Unable to add header with single digit zero

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





------- Additional Comments From jm@jmason.org  2006-09-29 11:02 -------
+1



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

[Bug 5089] Unable to add header with single digit zero

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


felicity@apache.org changed:

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






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

[Bug 5089] [review] Unable to add header with single digit zero

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


spamassassin@dostech.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|Unable to add header with   |[review] Unable to add
                   |single digit zero           |header with single digit
                   |                            |zero
  Status Whiteboard|                            |needs 2 votes




------- Additional Comments From spamassassin@dostech.ca  2006-09-18 05:59 -------
I don't see any way that the  || ""  is needed either.  The config parser
ensures the header value is present, so we should be good.

[dos@FC5-VPC trunk]$ svn ci -m "bug 5089: enable adding headers with single
digit zero value"
Sending        lib/Mail/SpamAssassin/PerMsgStatus.pm
Transmitting file data .
Committed revision 447269.




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

[Bug 5089] [review] Unable to add header with single digit zero

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


spamassassin@dostech.ca changed:

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




------- Additional Comments From spamassassin@dostech.ca  2006-10-03 15:58 -------
[dos@FC5-VPC 3.1]$ svn ci -m "bug 5089: enable adding headers with single digit
zero value"
Sending        lib/Mail/SpamAssassin/PerMsgStatus.pm
Transmitting file data .
Committed revision 452669.



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

[Bug 5089] [review] Unable to add header with single digit zero

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


spamassassin@dostech.ca changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|needs 2 votes               |needs 1 vote






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

[Bug 5089] [review] Unable to add header with single digit zero

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


sidney@sidney.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Status Whiteboard|needs 1 vote                |ready to commit




------- Additional Comments From sidney@sidney.com  2006-10-03 15:26 -------
+1



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