|
SA Bugzilla – Full Text Bug Listing |
Summary: | ASN plugin should run on URI's too | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Frank Ch. Eigler <fche> |
Component: | Plugins | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED WONTFIX | ||
Severity: | normal | CC: | apache |
Priority: | P2 | ||
Version: | 3.3.2 | ||
Target Milestone: | Undefined | ||
Hardware: | PC | ||
OS: | Linux | ||
Whiteboard: |
Description
Frank Ch. Eigler
2012-11-03 16:51:53 UTC
Not going to work for the reasons mentioned in Bug 5655. DNS lookup is required to lookup IP of URIs. Bayes is already tokenizing at that stage, it would have to be moved as the last thing in whole scan process to be able to use them. Also sa-learn would need using DNS then always for proper learning, since Bayes doesn't save the original tokens anywhere for reuse. Forcefeeding Bayes all kinds stuff doesn't necessarily make it better either, as proven many times. Closing this stale one. |