MirrorHub/synapse
0
0
Fork 1
mirror of https://mau.dev/maunium/synapse.git synced 2024-12-15 02:33:51 +01:00
Code Issues Projects Releases Wiki Activity

Consistency for how verify_request.deferred is called

Browse source 
Define that it is run with no log context, and make sure that happens.

If we aren't careful to reset the logcontext, we can't bung the deferreds into
defer.gatherResults etc. We don't actually do that directly, but we *do*
resolve other deferreds from affected callbacks (notably the server_to_deferred
map in _start_key_lookups), and those *do* get passed into
defer.gatherResults. It turns out that this way ends up being least confusing.
This commit is contained in:
Richard van der Hoff 2017-09-20 01:32:42 +01:00
parent 3b98439eca
commit 2a4b9ea233
1 changed files with 17 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
synapse/crypto/keyring.py
View file

@ -57,7 +57,8 @@ Attributes:
json_object(dict): The JSON object to verify. json_object(dict): The JSON object to verify.
deferred(twisted.internet.defer.Deferred): deferred(twisted.internet.defer.Deferred):
A deferred (server_name, key_id, verify_key) tuple that resolves when A deferred (server_name, key_id, verify_key) tuple that resolves when
a verify key has been fetched a verify key has been fetched. The deferreds' callbacks are run with no
logcontext.
""" """
@ -284,6 +285,7 @@ class Keyring(object):
if not missing_keys: if not missing_keys:
break break
with PreserveLoggingContext():
for verify_request in requests_missing_keys.values(): for verify_request in requests_missing_keys.values():
verify_request.deferred.errback(SynapseError( verify_request.deferred.errback(SynapseError(
401, 401,
@ -294,6 +296,7 @@ class Keyring(object):
)) ))
def on_err(err): def on_err(err):
with PreserveLoggingContext():
for verify_request in verify_requests: for verify_request in verify_requests:
if not verify_request.deferred.called: if not verify_request.deferred.called:
verify_request.deferred.errback(err) verify_request.deferred.errback(err)
@ -714,6 +717,7 @@ class Keyring(object):
def _handle_key_deferred(verify_request): def _handle_key_deferred(verify_request):
server_name = verify_request.server_name server_name = verify_request.server_name
try: try:
with PreserveLoggingContext():
_, key_id, verify_key = yield verify_request.deferred _, key_id, verify_key = yield verify_request.deferred
except IOError as e: except IOError as e:
logger.warn( logger.warn(