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

Refactor first sync with Synapse #7187

Open
fredo opened this issue Jun 24, 2021 · 1 comment
Open

Refactor first sync with Synapse #7187

fredo opened this issue Jun 24, 2021 · 1 comment
Labels
Component / Transport Transport related issues

Comments

@fredo
Copy link
Contributor

fredo commented Jun 24, 2021

The current implementation of initialize_first_sync() in MatrixTransport has the purpose to get the inventory of rooms for our partners. With the removal of rooms this is not needed anymore. Instead there might be a longer list of messages stored for the user which can contain lots of call messages for webrtc connection attempts.
Since the user will try to create a web rtc connection himself with its partners we can ignore those messages.
It is also possible to ignore all messages in the mailbox as we have the retry mechanism in place and Raiden protocol messages will be retried anyway.

  • Remove the filter of not fetching messages
  • perform /sync manually (as currently already done)
  • dump all the messages
@fredo fredo added the Component / Transport Transport related issues label Jun 24, 2021
@fredo fredo added this to Backlog in Raiden Berlin Sprint Jun 24, 2021
@fredo
Copy link
Contributor Author

fredo commented Jun 24, 2021

This is also partly covering #6824

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component / Transport Transport related issues
Projects
Development

No branches or pull requests

1 participant