Mailing List Archive: [Bug 5655] Bayes not considering add

[Bug 5655] Bayes not considering add_header'ed information

Apr 10, 2021, 2:43 AM

Post #1 of 3 (93 views)

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=5655

Henrik Krohns <apache@hege.li> changed:

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

--- Comment #10 from Henrik Krohns <apache@hege.li> ---

Bayes has access to ASN metadata now. Had to move processing from
parsed_metadata to extract_metadata, so it happens before Bayes tokenizes.

Sending trunk/lib/Mail/SpamAssassin/Plugin/ASN.pm
Sending trunk/lib/Mail/SpamAssassin.pm
Transmitting file data ..done
Committing transaction...
Committed revision 1888576.

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

[Bug 5655] Bayes not considering add_header'ed information [ In reply to ]

bugzilla-daemon at spamassassin

Apr 12, 2021, 12:25 PM

Post #2 of 3 (92 views)

Permalink

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=5655

--- Comment #11 from RW <rwmaillists@googlemail.com> ---
As I understand it the default is using a DNS look-up to get the ASN
information. So what happens when sa-learn runs? Does the look-up get repeated?

If it doesn't the ASN tokens will come only from auto-training with no
possibility of correction.

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

[Bug 5655] Bayes not considering add_header'ed information [ In reply to ]

bugzilla-daemon at spamassassin

Apr 12, 2021, 4:32 PM

Post #3 of 3 (92 views)

Permalink

https://bz.apache.org/SpamAssassin/show_bug.cgi?id=5655

--- Comment #12 from Henrik Krohns <apache@hege.li> ---
DNS is only used for ASN if GeoIP is not available.

In reality Bayes can't even use any DNS based data, since tokenizing is done
mere moments after async DNS is launched (dns prio -100, bayes -90). There is
no time to wait for actual replies.

Sa-learn doesn't even seem to harvest DNS replies, if some plugin decided to
query stuff. I guess all this was the reason for docs saying that network
lookups are not used even without -L. In the commit you see I blocked ASN from
sending DNS needlessly when learning, for some reason it was the only one that
did.

Now thinking about it, I guess sa-learn -L should be made a default option..
not sure what kind of network based test would even produce usable Bayes data.
ASN and any similar static data should be queried from local database.

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