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

Propagate MLS messages to remotes #2415

Merged
merged 16 commits into from
Jun 13, 2022

Conversation

pcapriotti
Copy link
Contributor

@pcapriotti pcapriotti commented May 19, 2022

Propagate MLS messages to remote conversation participants. This adds a new federation endpoint (send-mls-messages) and also restructures the function for constructing message pushes so that it does not need a member map anymore, only a map of bots.

This PR also adds an mls-clients column to the table containing remote members of a local conversation.

Tracked by https://wearezeta.atlassian.net/browse/FS-510

Checklist

  • The PR Title explains the impact of the change.
  • The PR description provides context as to why the change should occur and what the code contributes to that effect. This could also be a link to a JIRA ticket or a Github issue, if there is one.
  • If a cassandra schema migration has been added, I ran make git-add-cassandra-schema to update the cassandra schema documentation.
  • changelog.d contains the following bits of information (details):
    • A file with the changelog entry in one or more suitable sub-sections. The sub-sections are marked by directories inside changelog.d.

@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from ba01e2f to 8f6be82 Compare May 23, 2022 12:13
@pcapriotti pcapriotti temporarily deployed to cachix May 23, 2022 12:13 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from 8f6be82 to 89fc597 Compare May 23, 2022 12:40
@pcapriotti pcapriotti temporarily deployed to cachix May 23, 2022 12:40 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 23, 2022 14:06 Inactive
@mdimjasevic mdimjasevic temporarily deployed to cachix May 23, 2022 14:19 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from ee88046 to b4c2f88 Compare May 25, 2022 09:58
@pcapriotti pcapriotti temporarily deployed to cachix May 25, 2022 09:58 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch 2 times, most recently from b4c2f88 to 5f64d2a Compare May 27, 2022 08:29
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 08:29 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from 5f64d2a to a62b43f Compare May 27, 2022 08:33
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 08:33 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from a62b43f to 3533b49 Compare May 27, 2022 10:21
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 10:21 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 12:19 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from d27b769 to 29ed8c7 Compare May 27, 2022 13:28
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 13:28 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 27, 2022 13:53 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 30, 2022 07:12 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 30, 2022 09:58 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 31, 2022 08:43 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix May 31, 2022 12:18 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix June 1, 2022 08:58 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from efb8c97 to 165ba4e Compare June 1, 2022 09:40
@pcapriotti pcapriotti temporarily deployed to cachix June 1, 2022 09:45 Inactive
@pcapriotti pcapriotti force-pushed the pcapriotti/mls-propagate-remote-messages branch from 165ba4e to 96e7339 Compare June 1, 2022 12:15
@pcapriotti pcapriotti temporarily deployed to cachix June 1, 2022 12:15 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix June 1, 2022 15:45 Inactive
@pcapriotti pcapriotti marked this pull request as ready for review June 7, 2022 11:59
changelog.d/2-features/mls Outdated Show resolved Hide resolved
@mdimjasevic
Copy link
Contributor

Does CI need retriggering? It doesn't look like integration tests and other tests ran.

@pcapriotti pcapriotti temporarily deployed to cachix June 8, 2022 08:09 Inactive
@pcapriotti pcapriotti temporarily deployed to cachix June 9, 2022 13:35 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants