summary refs log tree commit diff
path: root/synapse/app
diff options
context:
space:
mode:
authorErik Johnston <erik@matrix.org>2020-12-10 16:56:05 +0000
committerGitHub <noreply@github.com>2020-12-10 16:56:05 +0000
commit80a992d7b953ea58dd45913d68855e396ad4d980 (patch)
tree449dea8edcd07a58907589ee1c316821b4603966 /synapse/app
parentRefactor `SsoHandler.get_mxid_from_sso` (#8900) (diff)
downloadsynapse-80a992d7b953ea58dd45913d68855e396ad4d980.tar.xz
Fix deadlock on SIGHUP (#8918)
Fixes #8892
Diffstat (limited to 'synapse/app')
-rw-r--r--synapse/app/_base.py6
1 files changed, 5 insertions, 1 deletions
diff --git a/synapse/app/_base.py b/synapse/app/_base.py
index 895b38ae76..37ecdbe3d8 100644
--- a/synapse/app/_base.py
+++ b/synapse/app/_base.py
@@ -245,6 +245,8 @@ def start(hs: "synapse.server.HomeServer", listeners: Iterable[ListenerConfig]):
         # Set up the SIGHUP machinery.
         if hasattr(signal, "SIGHUP"):
 
+            reactor = hs.get_reactor()
+
             @wrap_as_background_process("sighup")
             def handle_sighup(*args, **kwargs):
                 # Tell systemd our state, if we're using it. This will silently fail if
@@ -260,7 +262,9 @@ def start(hs: "synapse.server.HomeServer", listeners: Iterable[ListenerConfig]):
             # is so that we're in a sane state, e.g. flushing the logs may fail
             # if the sighup happens in the middle of writing a log entry.
             def run_sighup(*args, **kwargs):
-                hs.get_clock().call_later(0, handle_sighup, *args, **kwargs)
+                # `callFromThread` should be "signal safe" as well as thread
+                # safe.
+                reactor.callFromThread(handle_sighup, *args, **kwargs)
 
             signal.signal(signal.SIGHUP, run_sighup)