spamassassin-dev October 2011 archive
Main Archive Page > Month Archives  > spamassassin-dev archives
spamassassin-dev: [Bug 6413] sa-learn can't parse mbox of latest

[Bug 6413] sa-learn can't parse mbox of latest CommuniGate Pro

From: <bugzilla-daemon_at_nospam>
Date: Sat Oct 29 2011 - 02:41:05 GMT
To: dev@spamassassin.apache.org

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6413

--- Comment #2 from Kevin A. McGrail <kmcgrail@pccc.com> 2011-10-29 02:41:05 UTC ---
Created attachment 4995
  --> https://issues.apache.org/SpamAssassin/attachment.cgi?id=4995
Patch to change ArchiveIterator to also pick up Communigates Mbox format

Since the mbox format is very very unspecified (see
http://tools.ietf.org/html/rfc4155), expanding this a bit makes sense in a way.

However, changing the code as proposed in the PoC might break things for other
users. You were very clear it was just a proof of concept.

So I took your PoC and change it to a case in ArchiveIterator.pm that handles
both the existing and Communigate's format through the regex you provided.

It appears to work. I've tested with your example and an mbox file from mutt
via sendmail and procmail and believe it works interchangeably.

I will commit and would appreciate your feedback.

-- Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug.