Problem syncing mail - crash

[Sebastian Spaeth]

On Sat, 23 Jul 2011 15:48:20 +0200, Vladimir Marek <Vladimir.Marek at Oracle.COM> wrote:
> > I ran git bisect to find exactly when this started happening. It worked
> > in the most recent release (6.3.3), and stopped working with this
> > commit:
> > commit e20d8b967942934ddbf4659b5ec328a9a18da6bc
> > Author: Sebastian Spaeth <Sebastian at SSpaeth.de>
> > Date:   Thu Mar 24 17:45:21 2011 +0100
> > 
> >     Remove upload neguid pass from sync logic

Hi there, I am back from my holidays. I see that my "optimizatons" broke
something that used to work. Thanks for the bisection.

Perhaps I broke the behavior because I was ignorant of
that FAQ entry that states that we will delete the local Mail if the
server returns UID 0. :-)

I know it's the documented behavior, but is this really what we should
be doing here? If the server returns UID 0, something has gone wrong and is
bad. Perhaps the IMAP server responded "OK, I just deleted your uploaded
Mail MUAHAHAHAHA"
Should we really be deleting the local Maildir version of that mail
then? Only if we are really 100% sure that it uploaded correctly and
that we'll get it back on the next sync! Are we sufficiently certain and
happy with that strategy? Or do we want to risk duplicates rather than
potential data loss? I am not so sure.

Anyway, I guess the underlying reason why the search failed is that
offlineimap inserted the header at the wrong spot, and that was a real
bug (http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=482557). Thanks
for the other patch fixing it. I wonder though, if we shouldn't simply
throw the mail file at pythons email module and add the header via that
one, rather than manually fudging the mail content with
regexes. Personally, I'd feel more comfortable with that approach, even
if the performance might suffer (but then our bottleneck will always be
network performance anyway, so we might be fine).

Sebastian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.alioth.debian.org/pipermail/offlineimap-project/attachments/20110808/6efdab97/attachment.pgp>