summary refs log tree commit diff
path: root/synapse
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
parentRefactor `SsoHandler.get_mxid_from_sso` (#8900) (diff)
downloadsynapse-80a992d7b953ea58dd45913d68855e396ad4d980.tar.xz
Fix deadlock on SIGHUP (#8918)
Fixes #8892
Diffstat (limited to 'synapse')
-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)