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 2005/02/04 09:17:24 UTC

[Bug 4119] New: Bayes code should warn/die if bayes_path is malformed

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

           Summary: Bayes code should warn/die if bayes_path is malformed
           Product: Spamassassin
           Version: 3.0.2
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Libraries
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: quinlan@pathname.com


malformed - directory doesn't exist
          - is a directory rather than a partial filename under a directory

P.S. Why the heck isn't this a directory????



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

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119


jm@jmason.org changed:

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




------- Additional Comments From jm@jmason.org  2005-04-09 23:48 -------
good point; it'd need the trailing slash and that's really quite an unusual
thing to do accidentally. alright, I'm fine with that...



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119


jm@jmason.org changed:

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




------- Additional Comments From jm@jmason.org  2005-04-09 17:32 -------
fwiw, I note it now considers it invalid and does not use the setting.  

I'd prefer to warn about it, but use it anyway -- as some sickos may have used
it anyway (probably accidentally) since v2.x.  it'd be better in those cases to
not suddenly start failing to use their bayes db due to a (relatively minor)
misconfiguration.



------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119


quinlan@pathname.com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |dev@spamassassin.apache.org
         AssignedTo|dev@spamassassin.apache.org |quinlan@pathname.com




------- Additional Comments From quinlan@pathname.com  2005-04-03 00:46 -------
yo




------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
You are on the CC list for the bug, or are watching someone who is.

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119





------- Additional Comments From quinlan@pathname.com  2005-04-09 19:02 -------
Subject: Re:  Bayes code should warn/die if bayes_path is malformed


> fwiw, I note it now considers it invalid and does not use the setting.

> I'd prefer to warn about it, but use it anyway -- as some sickos may
> have used it anyway (probably accidentally) since v2.x.

Using a directory has never worked.  I'm -0.5 on hacking this so that a
directory works.

On second thought, "/foo/bar/" *with* a trailing slash might have worked
in 3.0, but I haven't tested it.





------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119


parkerm@pobox.com changed:

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






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

[Bug 4119] Bayes code should warn/die if bayes_path is malformed

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4119


quinlan@pathname.com changed:

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




------- Additional Comments From quinlan@pathname.com  2005-04-03 00:48 -------
I only changed bayes_path to consider directories an invalid value.

Warnings should be generalized somehow in the configuration parser.




------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.