diff options
| author | Richard van der Hoff <richard@matrix.org> | 2017-03-17 14:30:16 +0000 |
|---|---|---|
| committer | Richard van der Hoff <richard@matrix.org> | 2017-03-17 15:06:08 +0000 |
| commit | 0c01f829ae873bbc6451d5933ee77351ba20b93f (patch) | |
| tree | 9beddc25115cb9931933727b1ea6d7205999ce72 /synapse/storage/state.py | |
| parent | Refactoring and cleanups (diff) | |
| download | synapse-0c01f829ae873bbc6451d5933ee77351ba20b93f.tar.xz | |
Avoid resetting state on rejected events
When we get a rejected event, give it the same state_group as its prev_event, rather than no state_group at all. This should fix https://github.com/matrix-org/synapse/issues/1935.
Diffstat (limited to 'synapse/storage/state.py')
| -rw-r--r-- | synapse/storage/state.py | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/synapse/storage/state.py b/synapse/storage/state.py index 27f1ec89ec..1b42bea07a 100644 --- a/synapse/storage/state.py +++ b/synapse/storage/state.py @@ -136,6 +136,16 @@ class StateStore(SQLBaseStore): continue if context.current_state_ids is None: + # AFAIK, this can never happen + logger.error( + "Non-outlier event %s had current_state_ids==None", + event.event_id) + continue + + # if the event was rejected, just give it the same state as its + # predecessor. + if context.rejected: + state_groups[event.event_id] = context.prev_group continue state_groups[event.event_id] = context.state_group |