https://issues.apache.org/SpamAssassin/show_bug.cgi?id=5895

--- Comment #11 from Sidney Markowitz <[email protected]> 2010-01-05 06:01:20 
UTC ---
(in reply to comment 10)

I tried adding type => $CONF_TYPE_STRING, to originating_ip_headers but it
doesn't work because originating_ip_headers takes an array of strings, not a
string. Similarly, there is no constant to use for options like
clear_originating_ip_headers that take no argument.

As you suggested, we can create new types and eliminate the corresponding
entries in the list in t/cross_user_config_leak.t, ans I'm +1 for doing that,
but that's outside the scope of this bug.

For now, the right thing to do is to add originating_ip_headers and
clear_originating_ip_headers to the exception list in
t/cross_user_config_leak.t and deal with making new types in a separate bug.

With the change to t/cross_user_config_leak.t and Justin's suggested change to
the comments in comment #9 I'm +1 on committing this.

-- 
Configure bugmail: 
https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

Reply via email to