tobias/docker-offlineimap
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Don't embed CRLF multiple times when saving a message

Browse Source 
Since we just do multiple passes for saving the message without
actually modifying its content (apart from header insertion that
is CRLF-clean), we can change line ends to the proper CRLF just
once.

And we can also get message's date only once too.

Signed-off-by: Eygene Ryabinkin <rea@codelabs.ru>
This commit is contained in:
Eygene Ryabinkin 2014-05-02 15:14:22 +04:00
parent 0903428fda
commit 844ca6b08c
1 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
offlineimap/folder/IMAP.py
View File

@ -509,6 +509,12 @@ class IMAPFolder(BaseFolder):
self.savemessageflags(uid, flags)
return uid
# Use proper CRLF all over the message
content = re.sub("(?<!\r)\n", "\r\n", content)
# get the date of the message, so we can pass it to the server.
date = self.__getmessageinternaldate(content, rtime)
retry_left = 2 # succeeded in APPENDING?
imapobj = self.imapserver.acquireconnection()
# NB: in the finally clause for this try we will release
@ -521,10 +527,6 @@ class IMAPFolder(BaseFolder):
# UIDPLUS extension provides us with an APPENDUID response.
use_uidplus = 'UIDPLUS' in imapobj.capabilities
# get the date of the message, so we can pass it to the server.
date = self.__getmessageinternaldate(content, rtime)
content = re.sub("(?<!\r)\n", "\r\n", content)
if not use_uidplus:
# insert a random unique header that we can fetch later
(headername, headervalue) = self.__generate_randomheader(