diff options
| author | Brendan Abolivier <babolivier@matrix.org> | 2020-09-07 15:15:06 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-09-07 15:15:06 +0100 |
| commit | a55e2707d73c19b555c18ebc19df3170a96055e2 (patch) | |
| tree | 8773ce5694d0677d7af836fad5589e8fad70332e /synapse/storage | |
| parent | Add more logging to debug slow startup (#8264) (diff) | |
| download | synapse-a55e2707d73c19b555c18ebc19df3170a96055e2.tar.xz | |
Fix unread count failing on NULL values (#8270)
Fix unread counts making sync fail if the value of the `unread_count` column in `event_push_summary` is `None`.
Diffstat (limited to 'synapse/storage')
| -rw-r--r-- | synapse/storage/databases/main/event_push_actions.py | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/synapse/storage/databases/main/event_push_actions.py b/synapse/storage/databases/main/event_push_actions.py index 001d06378d..50fac9e72e 100644 --- a/synapse/storage/databases/main/event_push_actions.py +++ b/synapse/storage/databases/main/event_push_actions.py @@ -177,7 +177,12 @@ class EventPushActionsWorkerStore(SQLBaseStore): if row: notif_count += row[0] - unread_count += row[1] + + if row[1] is not None: + # The unread_count column of event_push_summary is NULLable, so we need + # to make sure we don't try increasing the unread counts if it's NULL + # for this row. + unread_count += row[1] return { "notify_count": notif_count, |