[PATCH] Re: Limit msg body length in debug output
Sebastian at SSpaeth.de
Mon May 9 22:34:39 BST 2011
On Mon, 9 May 2011 21:23:40 +0200, Nicolas Sebrecht wrote:
> True. It's exactly what I reported back in this mail:
Yes, I think so too.
> What we should probably do is to catch this
> error and raise a OfflineImapError with severity MESSAGE. As previously
> said, there two reasons I can see why we get this error:
> - the mail is actually deleted on remote;
> - something was going wrong while fetching it.
I think we can differentiate between those case. When somthing went
wrong while fetching, we won't get the "OK" reply (implying the server
is finished) but it will crap out in the "fetch" part already.
We can detect that the mail has been deleted by checking that the data
variable is empty (need to verify this).
Either we return "None", or we raise an OfflineImapError, severity
Message, both makes sense. Whatever we do, we need to make sure the
Maildir backend returns the same, in case a message has been deleted
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 197 bytes
Desc: not available
More information about the OfflineIMAP-project