Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Synchrotron #48

Open
wants to merge 20 commits into
base: master
from

Conversation

Projects
None yet
1 participant
@Half-Shot
Copy link
Collaborator

commented Mar 24, 2019

This is a implementation of a sync worker. It supports /sync, /rooms/.../initialsync, and it's friends /rooms/.../(members|messages|context). These are required for /sync to work with lazy loading.

It does NOT support the legacy endpoints /events or /initialsync.

Bugs:

  • Unread room highlights are wrong, even after reading.

    • I suspect this is because we give the wrong ordering to Riot, so it "reads" the wrong message.
  • Joining rooms seems to leave me with sketchy history #matrix-dev #synapse-netcore-workers

    • Also an ordering bug, because we backfill after joining which means events will arrive in a random order.
  • Message ordering isn't as reliable as one would like

  • We don't support groups at all yet, cause yuck. We should probably do that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.