diff options
author | Mark Haines <mark.haines@matrix.org> | 2016-08-03 11:12:47 +0100 |
---|---|---|
committer | Mark Haines <mark.haines@matrix.org> | 2016-08-03 11:12:47 +0100 |
commit | 921f17f93810ac07fd5d15494882db78fdcf3e3c (patch) | |
tree | bd4f29ec888a4e445654688e0f6da994be899a63 /synapse/push/emailpusher.py | |
parent | E2E keys: Make federation query share code with client query (diff) | |
parent | Factor out some of the code shared between the sytest scripts (#974) (diff) | |
download | synapse-921f17f93810ac07fd5d15494882db78fdcf3e3c.tar.xz |
Merge branch 'develop' into rav/refactor_device_query
Diffstat (limited to 'synapse/push/emailpusher.py')
-rw-r--r-- | synapse/push/emailpusher.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/synapse/push/emailpusher.py b/synapse/push/emailpusher.py index e224b68291..6600c9cd55 100644 --- a/synapse/push/emailpusher.py +++ b/synapse/push/emailpusher.py @@ -14,6 +14,7 @@ # limitations under the License. from twisted.internet import defer, reactor +from twisted.internet.error import AlreadyCalled, AlreadyCancelled import logging @@ -92,7 +93,11 @@ class EmailPusher(object): def on_stop(self): if self.timed_call: - self.timed_call.cancel() + try: + self.timed_call.cancel() + except (AlreadyCalled, AlreadyCancelled): + pass + self.timed_call = None @defer.inlineCallbacks def on_new_notifications(self, min_stream_ordering, max_stream_ordering): @@ -189,7 +194,10 @@ class EmailPusher(object): soonest_due_at = should_notify_at if self.timed_call is not None: - self.timed_call.cancel() + try: + self.timed_call.cancel() + except (AlreadyCalled, AlreadyCancelled): + pass self.timed_call = None if soonest_due_at is not None: |