diff options
| author | Jorik Schellekens <joriks@matrix.org> | 2019-07-15 15:23:33 +0100 |
|---|---|---|
| committer | Jorik Schellekens <joriks@matrix.org> | 2019-07-23 16:05:58 +0100 |
| commit | ab191f99f9ab3a4308054a29a8693f42c9a3f3cc (patch) | |
| tree | 97b692bdce580a494afa8d7f440513dc279b08d0 | |
| parent | Though style is subjective it depends on a ruthless objectivity: you either h... (diff) | |
| download | synapse-ab191f99f9ab3a4308054a29a8693f42c9a3f3cc.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 cd5ab70d56..87fb302bf1 100644 --- a/synapse/handlers/device.py +++ b/synapse/handlers/device.py @@ -594,16 +594,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"] |