Simplified Sliding Sync #28515
Merged
Simplified Sliding Sync #28515
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Working branch to get SSS functional on element-web. Requires matrix-org/matrix-js-sdk#4400
This is actually required for SSS because otherwise it would use the proxy over native support.
- Update the spidering code to spider all the relevant lists. - Add canonical alias to the required_state to allow room name calcs to work. Room sort order is busted because we don't yet look at `bump_stamp`.
|
|
24 tasks
as they're not displayed anywhere so not useful
t3chguy
requested changes
Feb 20, 2025
by making it not a feature
and minus the bit about disabling notification which surely would have defeated the original point anyway?
...and also make tests actually use sliding sync
t3chguy
approved these changes
Mar 10, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Replaces the sliding sync option in labs with Simplified Sliding Sync so it will support the impl in Synapse directly rather than via the sliding sync proxy.
Anyone who had the old labs flag turned on will find that their client breaks very abruptly when updated: it will simply throw an error when it tries to start syncing. This is done deliberately to fail fast since otherwise it could cause subtle crypto failures with both a proxy and the client syncing directly. matrix.org has not run a sliding sync proxy for some time but other homeservers may still be running one.
This is still a way from perfect but getting it into labs is a place where we can start getting it ready for primetime.
Requires matrix-org/matrix-js-sdk#4400
Ported from element-hq/matrix-react-sdk#36
Checklist
public/exportedsymbols have accurate TSDoc documentation.