summary refs log tree commit diff
path: root/synapse/util/__init__.py
diff options
context:
space:
mode:
authorTravis Ralston <travpc@gmail.com>2018-10-12 16:24:07 -0600
committerTravis Ralston <travpc@gmail.com>2018-10-12 16:24:07 -0600
commit7ede6509561ebe4c217210ea3431421fcf9910b2 (patch)
tree146a92462af13f48ca90f5096394d34b573eb393 /synapse/util/__init__.py
parentChangelog (diff)
parentMake workers work on Py3 (#4027) (diff)
downloadsynapse-7ede6509561ebe4c217210ea3431421fcf9910b2.tar.xz
Merge branch 'develop' into travis/login-terms
Diffstat (limited to 'synapse/util/__init__.py')
-rw-r--r--synapse/util/__init__.py31
1 files changed, 30 insertions, 1 deletions
diff --git a/synapse/util/__init__.py b/synapse/util/__init__.py
index 680ea928c7..9a8fae0497 100644
--- a/synapse/util/__init__.py
+++ b/synapse/util/__init__.py
@@ -68,7 +68,10 @@ class Clock(object):
         """
         call = task.LoopingCall(f)
         call.clock = self._reactor
-        call.start(msec / 1000.0, now=False)
+        d = call.start(msec / 1000.0, now=False)
+        d.addErrback(
+            log_failure, "Looping call died", consumeErrors=False,
+        )
         return call
 
     def call_later(self, delay, callback, *args, **kwargs):
@@ -109,3 +112,29 @@ def batch_iter(iterable, size):
     sourceiter = iter(iterable)
     # call islice until it returns an empty tuple
     return iter(lambda: tuple(islice(sourceiter, size)), ())
+
+
+def log_failure(failure, msg, consumeErrors=True):
+    """Creates a function suitable for passing to `Deferred.addErrback` that
+    logs any failures that occur.
+
+    Args:
+        msg (str): Message to log
+        consumeErrors (bool): If true consumes the failure, otherwise passes
+            on down the callback chain
+
+    Returns:
+        func(Failure)
+    """
+
+    logger.error(
+        msg,
+        exc_info=(
+            failure.type,
+            failure.value,
+            failure.getTracebackObject()
+        )
+    )
+
+    if not consumeErrors:
+        return failure