Re: [Mailman-Developers] Possible bug in marilman arch when doing incremental archiving

On 12/13/2013 05:06 AM, fc lists wrote:
When the start is higher then counter, to iterate over the mailbox and get to the starting point requested, the *next()* method on the mbox object is called which trigger the archfactory which extract the attachment from messages that were already archived causing the duplicate attachment to happen.
My "test fix" is basically to set the factory for the mbox object to the default rfc822.Message while iterating over the mbox object until the start is reached , then i set the factory back to the _archfactory ... and everything seems to work as expected.
Thank you for the report. I have created https://bugs.launchpad.net/mailman/+bug/1260883 for this issue.
I think your fix is OK, but I am inclined to set the factory while skipping to the start to Mailman.Mailbox._safeparser rather than the default just in case one of the skipped messages throws MessageParseError.

On 12/13/2013 02:57 PM, Mark Sapiro wrote:
Thank you for the report. I have created https://bugs.launchpad.net/mailman/+bug/1260883 for this issue.
I think your fix is OK, but I am inclined to set the factory while skipping to the start to Mailman.Mailbox._safeparser rather than the default just in case one of the skipped messages throws MessageParseError.
I have committed a fix. I fixed this in a way that is probably different from what you did because I didn't want to import private Mailman.Mailbox functions into Mailman/Archiver/pipermail.py.
See http://bazaar.launchpad.net/~mailman-coders/mailman/2.1/revision/1440 for my fix.

Thanks a lot for this :)
I will take a look at your fix and most likely import that in my mailman package ... other then anything, i would prefer to have something mantained upstream so i don't have to patch my package every time i upgrade.
On Sat, Dec 14, 2013 at 1:09 AM, Mark Sapiro mark@msapiro.net wrote:
On 12/13/2013 02:57 PM, Mark Sapiro wrote:
Thank you for the report. I have created https://bugs.launchpad.net/mailman/+bug/1260883 for this issue.
I think your fix is OK, but I am inclined to set the factory while skipping to the start to Mailman.Mailbox._safeparser rather than the default just in case one of the skipped messages throws
MessageParseError.
I have committed a fix. I fixed this in a way that is probably different from what you did because I didn't want to import private Mailman.Mailbox functions into Mailman/Archiver/pipermail.py.
See http://bazaar.launchpad.net/~mailman-coders/mailman/2.1/revision/1440 for my fix.
-- Mark Sapiro mark@msapiro.net The highway is for gamblers, San Francisco Bay Area, California better use your sense - B. Dylan _______________________________________________ Mailman-Developers mailing list Mailman-Developers@python.org https://mail.python.org/mailman/listinfo/mailman-developers Mailman FAQ: http://wiki.list.org/x/AgA3 Searchable Archives: http://www.mail-archive.com/mailman-developers%40python.org/ Unsubscribe: https://mail.python.org/mailman/options/mailman-developers/fclists%40pr-z.in...
Security Policy: http://wiki.list.org/x/QIA9
participants (2)
-
fc lists
-
Mark Sapiro