summary refs log tree commit diff
path: root/synapse/handlers
diff options
context:
space:
mode:
authorErik Johnston <erikj@jki.re>2018-10-09 10:12:10 +0100
committerGitHub <noreply@github.com>2018-10-09 10:12:10 +0100
commit9eb1a7910020cbe867cb9b89414e95a9e342a81b (patch)
tree849d18757759d02f65c7e5c186876fad8d8ee947 /synapse/handlers
parentMerge pull request #4017 from matrix-org/rav/optimise_filter_events_for_server (diff)
parentRemove unnecessary extra function call layer (diff)
downloadsynapse-9eb1a7910020cbe867cb9b89414e95a9e342a81b.tar.xz
Merge pull request #4008 from matrix-org/erikj/log_looping_exceptions
Log looping call exceptions
Diffstat (limited to 'synapse/handlers')
-rw-r--r--synapse/handlers/appservice.py17
1 files changed, 5 insertions, 12 deletions
diff --git a/synapse/handlers/appservice.py b/synapse/handlers/appservice.py
index f0f89af7dc..17eedf4dbf 100644
--- a/synapse/handlers/appservice.py
+++ b/synapse/handlers/appservice.py
@@ -28,6 +28,7 @@ from synapse.metrics import (
     event_processing_loop_room_count,
 )
 from synapse.metrics.background_process_metrics import run_as_background_process
+from synapse.util import log_failure
 from synapse.util.logcontext import make_deferred_yieldable, run_in_background
 from synapse.util.metrics import Measure
 
@@ -36,17 +37,6 @@ logger = logging.getLogger(__name__)
 events_processed_counter = Counter("synapse_handlers_appservice_events_processed", "")
 
 
-def log_failure(failure):
-    logger.error(
-        "Application Services Failure",
-        exc_info=(
-            failure.type,
-            failure.value,
-            failure.getTracebackObject()
-        )
-    )
-
-
 class ApplicationServicesHandler(object):
 
     def __init__(self, hs):
@@ -112,7 +102,10 @@ class ApplicationServicesHandler(object):
 
                         if not self.started_scheduler:
                             def start_scheduler():
-                                return self.scheduler.start().addErrback(log_failure)
+                                return self.scheduler.start().addErrback(
+                                    log_failure, "Application Services Failure",
+                                )
+
                             run_as_background_process("as_scheduler", start_scheduler)
                             self.started_scheduler = True