From 48eea2554803fa18f745d0e9234d950b3da5dbcf Mon Sep 17 00:00:00 2001
From: Folker Bernitt <fbernitt@thoughtworks.com>
Date: Mon, 25 Jan 2016 17:48:39 +0100
Subject: Fix: Ignore missing mail data error when creating search index

- Should fix accounts that can currently not log in because of mails
  where part of the docs are missing
- Does only fix the symptoms
---
 service/pixelated/adapter/mailstore/leap_mailstore.py | 15 +++++++++++----
 service/pixelated/adapter/services/mail_service.py    |  2 +-
 2 files changed, 12 insertions(+), 5 deletions(-)

(limited to 'service/pixelated/adapter')

diff --git a/service/pixelated/adapter/mailstore/leap_mailstore.py b/service/pixelated/adapter/mailstore/leap_mailstore.py
index 4a0de6a1..9b62b3ba 100644
--- a/service/pixelated/adapter/mailstore/leap_mailstore.py
+++ b/service/pixelated/adapter/mailstore/leap_mailstore.py
@@ -21,6 +21,7 @@ from uuid import uuid4
 from leap.mail.adaptors.soledad import SoledadMailAdaptor
 from leap.mail.mail import Message
 from twisted.internet import defer
+from twisted.internet.defer import FirstError, DeferredList
 
 from pixelated.adapter.mailstore.body_parser import BodyParser
 from pixelated.adapter.mailstore.mailstore import MailStore, underscore_uuid
@@ -203,12 +204,18 @@ class LeapMailStore(MailStore):
 
         defer.returnValue(leap_mail)
 
-    def get_mails(self, mail_ids):
+    @defer.inlineCallbacks
+    def get_mails(self, mail_ids, gracefully_ignore_errors=False):
         deferreds = []
         for mail_id in mail_ids:
             deferreds.append(self.get_mail(mail_id, include_body=True))
 
-        return defer.gatherResults(deferreds, consumeErrors=True)
+        if gracefully_ignore_errors:
+            results = yield DeferredList(deferreds, consumeErrors=True)
+            defer.returnValue([mail for ok, mail in results if ok and mail is not None])
+        else:
+            result = yield defer.gatherResults(deferreds, consumeErrors=True)
+            defer.returnValue(result)
 
     @defer.inlineCallbacks
     def update_mail(self, mail):
@@ -218,12 +225,12 @@ class LeapMailStore(MailStore):
         yield self._update_mail(message)  # TODO assert this is yielded (otherwise asynchronous)
 
     @defer.inlineCallbacks
-    def all_mails(self):
+    def all_mails(self, gracefully_ignore_errors=False):
         mdocs = yield self.soledad.get_from_index('by-type', 'meta')
 
         mail_ids = map(lambda doc: doc.doc_id, mdocs)
 
-        mails = yield self.get_mails(mail_ids)
+        mails = yield self.get_mails(mail_ids, gracefully_ignore_errors=gracefully_ignore_errors)
         defer.returnValue(mails)
 
     @defer.inlineCallbacks
diff --git a/service/pixelated/adapter/services/mail_service.py b/service/pixelated/adapter/services/mail_service.py
index cf47bfa6..bfe45cad 100644
--- a/service/pixelated/adapter/services/mail_service.py
+++ b/service/pixelated/adapter/services/mail_service.py
@@ -37,7 +37,7 @@ class MailService(object):
 
     @defer.inlineCallbacks
     def all_mails(self):
-        mails = yield self.mail_store.all_mails()
+        mails = yield self.mail_store.all_mails(gracefully_ignore_errors=True)
         defer.returnValue(mails)
 
     def save_attachment(self, content, content_type):
-- 
cgit v1.2.3