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

Tree in GroupInfo not optional #344

Closed
raphaelrobert opened this issue Jun 2, 2020 · 0 comments · Fixed by #348
Closed

Tree in GroupInfo not optional #344

raphaelrobert opened this issue Jun 2, 2020 · 0 comments · Fixed by #348

Comments

@raphaelrobert
Copy link
Member

I remember we discussed this in the past, but I don't think we ever spec'ed this out fully:

Right now the whole tree is part of GroupInfo (which in turn is part of Welcome). This can grow quite big in large groups (~17MB with 100k members with the MTI). The practical limitations are:

  • The TLS representation of the tree is hard to parse efficiently (no skipping and no a priori information of how many nodes are in the tree)
  • Out-of-band transmission (like server assist) is not possible
  • Large messages like that are hard to handle on mobile devices, when they have to be downloaded in the background

We should introduce an indirection, like a commit hash of the tree.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

1 participant