summary refs log tree commit diff
path: root/synapse/push
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2019-02-22 15:18:19 +0000
committerErik Johnston <erik@matrix.org>2019-02-22 15:18:19 +0000
commitf2891d248756c968c346df052c796c0c69d6e764 (patch)
treeb75a30c3f6f67045eabb362c8029f42836824dc5 /synapse/push
parentNewsfile (diff)
downloadsynapse-f2891d248756c968c346df052c796c0c69d6e764.tar.xz
Correctly handle PusherConfigException
Diffstat (limited to 'synapse/push')
-rw-r--r--synapse/push/pusherpool.py9
1 files changed, 9 insertions, 0 deletions
diff --git a/synapse/push/pusherpool.py b/synapse/push/pusherpool.py
index b2c92ce683..af82e02b46 100644
--- a/synapse/push/pusherpool.py
+++ b/synapse/push/pusherpool.py
@@ -19,6 +19,7 @@ import logging
 from twisted.internet import defer
 
 from synapse.metrics.background_process_metrics import run_as_background_process
+from synapse.push import PusherConfigException
 from synapse.push.pusher import PusherFactory
 
 logger = logging.getLogger(__name__)
@@ -222,6 +223,14 @@ class PusherPool:
         """
         try:
             p = self.pusher_factory.create_pusher(pusherdict)
+        except PusherConfigException as e:
+            logger.warning(
+                "Pusher incorrectly configured user=%s, appid=%s, pushkey=%s: %s",
+                pusherdict.get('user_name'),
+                pusherdict.get('app_id'),
+                pusherdict.get('pushkey'),
+                e,
+            )
         except Exception:
             logger.exception("Couldn't start a pusher: caught Exception")
             return