diff options
| author | David Robertson <davidr@element.io> | 2023-03-22 12:53:55 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-03-22 12:53:55 +0000 |
| commit | 1bc9985eb7feca2a8eb4a0125cd03dfa3ac631fe (patch) | |
| tree | 4e54f44cb2f852316af725519e6797fdd477cbb8 /changelog.d/15309.bugfix | |
| parent | Change the parameter `immediate` of `send_device_messages` to default to `Tru... (diff) | |
| download | synapse-1bc9985eb7feca2a8eb4a0125cd03dfa3ac631fe.tar.xz | |
Have replication clients remove _INT_STREAM_POS (#15309)
* Have replication clients remove _INT_STREAM_POS Suppose worker A makes an internal http request from worker B. B may make changes that A later learns about over replication. We want A's request to block until it has seen those changes—mainly to ensure A's caches are invalidated promptly. This helps provide read-after-write consistency, eliminating entire categories of races and test flakes. To implement this, B includes a top-level field `_INT_STREAM_POS` in its response JSON. Roughly speaking, the field's value tells A what to wait for. But we weren't removing that internal field before A's request completed! Introduced in https://github.com/matrix-org/synapse/pull/14820. Fixes #15308. * Changelog
Diffstat (limited to 'changelog.d/15309.bugfix')
| -rw-r--r-- | changelog.d/15309.bugfix | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/changelog.d/15309.bugfix b/changelog.d/15309.bugfix new file mode 100644 index 0000000000..4d3fe4e4b1 --- /dev/null +++ b/changelog.d/15309.bugfix @@ -0,0 +1 @@ +Fix a bug introduced in Synapse 1.76.0 where responses from worker deployments could include an internal `_INT_STREAM_POS` key. |