[Bug 3609] New: spamd doesn't allow bayes_learn_to_journal

[bu...@bugzilla.spamassassin.org]

http://bugzilla.spamassassin.org/show_bug.cgi?id=3609

           Summary: spamd doesn't allow bayes_learn_to_journal
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P5
         Component: spamc/spamd
        AssignedTo: spamassassin-dev@incubator.apache.org
        ReportedBy: felicity@kluge.net


I was noticing that the code all looks at main->{learn_to_journal} which is set by init() and possibly 
init_learner().  init() is called with the global config only, then spamd forks.  init_learner() is never called 
with the value of the user's bayes_learn_to_journal setting, so it's not used.

two options imo:

1) make bayes_learn_to_journal a priv option, thereby making the issue moot.  not a bad idea since 
bayes_path is admin required, so don't want users able to flip between journal and db writes.

2) add an init_learner() call after the user config is read to reset appropriately.  backup the global 
version in spamd.


of these, I like 1.  thoughts?



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