diff options
author | Erik Johnston <erik@matrix.org> | 2020-12-10 16:56:05 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-12-10 16:56:05 +0000 |
commit | 80a992d7b953ea58dd45913d68855e396ad4d980 (patch) | |
tree | 449dea8edcd07a58907589ee1c316821b4603966 /synapse | |
parent | Refactor `SsoHandler.get_mxid_from_sso` (#8900) (diff) | |
download | synapse-80a992d7b953ea58dd45913d68855e396ad4d980.tar.xz |
Fix deadlock on SIGHUP (#8918)
Fixes #8892
Diffstat (limited to 'synapse')
-rw-r--r-- | synapse/app/_base.py | 6 |
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) |