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.

-- 
Mark Sapiro <m...@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/archive%40jab.org

Security Policy: http://wiki.list.org/x/QIA9

Reply via email to