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

--- Comment #6 from Kevin A. McGrail <[email protected]> 2011-06-28 13:56:05 
UTC ---
Additional problems (recapping as many as I know about from discussions on
dev):

1 - 70_sandbox.cf is not correct from the fixes in 6297 which leave rules out
of masscheck

2 - remnants of rules that are #testrules with no publish still get published.

3 - If it doesn't break masschecks, we should consider code to avoid publishing
ANY rules T_ and also add a safety valve parameter that requires a --test-rules
to run them even if they are published.

4 - According to http://wiki.apache.org/spamassassin/SaUpdateBackend, T_ rules
are never published and this page needs to be updated to document #testrules

5 - There are rules like T_SURBL_MULTI2 that are in mkettler's sandbox.  There
is currently no reason I can see for why these are being published.  This is
NOT an error in his sandbox.

6 - There might be race conditions where items on SaUpdateBackend flag for
always publish and never publish.  Which takes precedence?  I clearly thing NOT
publish should be the default but these conditions need to be clarified and the
code changed to reflect the conditions so that we never have this messed up
again.

-- 
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