diff options
| author | Jorik Schellekens <joriks@matrix.org> | 2019-07-15 15:23:33 +0100 |
|---|---|---|
| committer | Jorik Schellekens <joriks@matrix.org> | 2019-07-17 14:33:00 +0100 |
| commit | 0e8c35c8e8cda4fdcd370edbe4d9da220df60547 (patch) | |
| tree | d237ed8c74919ee34701c94c3b92d47160dfd562 | |
| parent | Though style is subjective it depends on a ruthless objectivity: you either h... (diff) | |
| download | synapse-0e8c35c8e8cda4fdcd370edbe4d9da220df60547.tar.xz | |
A little extra device_list tracing
| -rw-r--r-- | synapse/handlers/device.py | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/synapse/handlers/device.py b/synapse/handlers/device.py index 9bd42ed819..878296da78 100644 --- a/synapse/handlers/device.py +++ b/synapse/handlers/device.py @@ -597,16 +597,25 @@ class DeviceListEduUpdater(object): # eventually become consistent. return except FederationDeniedError as e: + opentracing.set_tag("error", True) + opentracing.log_kv({"reason": "FederationDeniedError"}) logger.info(e) return - except Exception: + except Exception as e: # TODO: Remember that we are now out of sync and try again # later + opentracing.set_tag("error", True) + opentracing.log_kv( + { + "message": "Exception raised by federation request", + "exception": e, + } + ) logger.exception( "Failed to handle device list update for %s", user_id ) return - + opentracing.log_kv({"result": result}) stream_id = result["stream_id"] devices = result["devices"] |