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 /changelog.d | |
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 'changelog.d')
-rw-r--r-- | changelog.d/7158.misc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/7158.misc b/changelog.d/7158.misc new file mode 100644 index 0000000000..269b8daeb0 --- /dev/null +++ b/changelog.d/7158.misc @@ -0,0 +1 @@ +Fix device list update stream ids going backward. |