diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2019-11-25 13:26:59 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-11-25 13:26:59 +0000 |
| commit | b7367c339db153824fb47728d3eebe2f944530e6 (patch) | |
| tree | 9efb1b5f09c12cecbed2bfbd32f801e5affd1f74 /synapse | |
| parent | 1.6.0rc1 (diff) | |
| download | synapse-b7367c339db153824fb47728d3eebe2f944530e6.tar.xz | |
Fix exceptions from background database update for event labels. (#6407)
Add some exception handling here so that events whose json cannot be parsed are ignored rather than getting us stuck in a loop. Fixes #6404.
Diffstat (limited to 'synapse')
| -rw-r--r-- | synapse/storage/data_stores/main/events_bg_updates.py | 43 |
1 files changed, 25 insertions, 18 deletions
diff --git a/synapse/storage/data_stores/main/events_bg_updates.py b/synapse/storage/data_stores/main/events_bg_updates.py index 0ed59ef48e..aa87f9abc5 100644 --- a/synapse/storage/data_stores/main/events_bg_updates.py +++ b/synapse/storage/data_stores/main/events_bg_updates.py @@ -530,24 +530,31 @@ class EventsBackgroundUpdatesStore(BackgroundUpdateStore): nbrows = 0 last_row_event_id = "" for (event_id, event_json_raw) in results: - event_json = json.loads(event_json_raw) - - self._simple_insert_many_txn( - txn=txn, - table="event_labels", - values=[ - { - "event_id": event_id, - "label": label, - "room_id": event_json["room_id"], - "topological_ordering": event_json["depth"], - } - for label in event_json["content"].get( - EventContentFields.LABELS, [] - ) - if isinstance(label, str) - ], - ) + try: + event_json = json.loads(event_json_raw) + + self._simple_insert_many_txn( + txn=txn, + table="event_labels", + values=[ + { + "event_id": event_id, + "label": label, + "room_id": event_json["room_id"], + "topological_ordering": event_json["depth"], + } + for label in event_json["content"].get( + EventContentFields.LABELS, [] + ) + if isinstance(label, str) + ], + ) + except Exception as e: + logger.warning( + "Unable to load event %s (no labels will be imported): %s", + event_id, + e, + ) nbrows += 1 last_row_event_id = event_id |