diff options
| author | Richard van der Hoff <1389908+richvdh@users.noreply.github.com> | 2020-04-03 10:40:22 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-04-03 10:40:22 +0100 |
| commit | daa1ac89a0be4dd3cc941da4caeb2ddcbd701eff (patch) | |
| tree | 58ee94b0cfead33ff1aa6cb7420873db994a953c /tests/util | |
| parent | Merge branch 'master' into develop (diff) | |
| download | synapse-daa1ac89a0be4dd3cc941da4caeb2ddcbd701eff.tar.xz | |
Fix device list update stream ids going backward (#7158)
Occasionally we could get a federation device list update transaction which
looked like:
```
[
{'edu_type': 'm.device_list_update', 'content': {'user_id': '@user:test', 'device_id': 'D2', 'prev_id': [], 'stream_id': 12, 'deleted': True}},
{'edu_type': 'm.device_list_update', 'content': {'user_id': '@user:test', 'device_id': 'D1', 'prev_id': [12], 'stream_id': 11, 'deleted': True}},
{'edu_type': 'm.device_list_update', 'content': {'user_id': '@user:test', 'device_id': 'D3', 'prev_id': [11], 'stream_id': 13, 'deleted': True}}
]
```
Having `stream_ids` which are lower than `prev_ids` looks odd. It might work
(I'm not actually sure), but in any case it doesn't seem like a reasonable
thing to expect other implementations to support.Diffstat (limited to 'tests/util')
0 files changed, 0 insertions, 0 deletions