-
Notifications
You must be signed in to change notification settings - Fork 333
Sending MLS messages to remote conversations #2378
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
Merged
Merged
Conversation
This file contains hidden or 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
2873ec4
to
d263be9
Compare
d263be9
to
9057378
Compare
d5f1a8a
to
99ccf9a
Compare
99ccf9a
to
94b7b8c
Compare
f4ed539
to
ddfa4f7
Compare
992b430
to
567b1d7
Compare
165ba4e
to
96e7339
Compare
567b1d7
to
37d0356
Compare
4c85b52
to
b7953fb
Compare
37d0356
to
bd4fc91
Compare
6349e1d
to
8a97ba3
Compare
Base automatically changed from
pcapriotti/mls-propagate-remote-messages
to
develop
June 13, 2022 06:46
c01e8c2
to
153d5b7
Compare
09b5d33
to
b8a8624
Compare
3b8dee8
to
c148380
Compare
Requesting domains do not receive notifications for leave events anymore.
smatting
approved these changes
Jul 8, 2022
stefanwire
approved these changes
Jul 8, 2022
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.
This PR implements sending remote messages to remote MLS conversations. This
involves a new RPC
send-mls-message
, similar tosend-message
for Proteus.Additionally, since remote backends need to set up a conversation ID / group ID
mapping for remote conversations, in order to know where to forward messages to,
a new RPC
on-new-remote-conversation
has been added. This is invoked the firsttime a user from a given remote backend is added to a conversation, and gives
that backend a chance to set up the corresponding mapping.
Tracked by https://wearezeta.atlassian.net/browse/FS-511.
Checklist
changelog.d
.