Bug 5468 - shortcircuit always has autolearn=disabled on spam messages
Summary: shortcircuit always has autolearn=disabled on spam messages
Status: RESOLVED WONTFIX
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Learner (show other bugs)
Version: 3.2.0
Hardware: PC Linux
: P5 normal
Target Milestone: Undefined
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-05-17 21:07 UTC by Mark Beeson
Modified: 2007-08-02 05:23 UTC (History)
1 user (show)


Attachment Type Modified Status Actions Submitter/CLA Status
Sample shortcircuit configuration and log line from the resulting config. text/plain None Mark Beeson [NoCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Mark Beeson 2007-05-17 21:07:20 UTC 
Regardless of how tflags is set to a shortcircuit rule, the autolearn never
learns a message as spam.

See attached for configuration and log line sample.
Comment 1 Mark Beeson 2007-05-17 21:10:56 UTC 
Created attachment 3947 [details]
Sample shortcircuit configuration and log line from the resulting config.
Comment 2 Michael Parker 2007-05-17 21:47:23 UTC 
I'd have to look to be 100% sure but I believe this is expected behavior. 
Because not all of the rules ran it is not possible to determine if a message
should be autolearned or not, so autolearning is disabled.
Comment 3 Mark Beeson 2007-05-17 22:00:20 UTC 
http://spamassassin.apache.org/full/3.2.x/doc/Mail_SpamAssassin_Plugin_Shortcircuit.html

...the documentation is a little vague on this.  Setting shortcircuit to spam
assumes tflags noautolearn, but I want to learn these messages as spam.  This is
why my config is only setting shortcircuit to on, and tflags to net.
Comment 4 Michael Parker 2007-05-17 22:14:01 UTC 
Yep, it gets disabled.  Bug 5256.  I agree that the documentation seems a little
confusing, but checking the code confirms that whenever a shortcircuit rule hits
auto learning is turned off.