Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@codabrink
Copy link
Contributor

@codabrink codabrink commented Jun 25, 2025

Add welcome metadata by restructuring WelcomeMessage.V1 and WelcomeMessageInput.V1 to use WelcomeMetadata message for message cursor field in MLS protocol buffer definition

The MLS protocol buffer definition restructures how message cursors are handled in welcome messages by moving the message_cursor field from direct inclusion in WelcomeMessage.V1 and WelcomeMessageInput.V1 to a new WelcomeMetadata message structure. The changes modify field 7 in WelcomeMessage.V1 from uint64 message_cursor to bytes welcome_metadata, update WelcomeMessageInput.V1 by changing field 5 from uint64 message_cursor to bytes welcome_metadata at field 7, and introduce the new WelcomeMetadata message containing the uint64 message_cursor field that gets encrypted alongside the data field. See mls.proto.

πŸ“Where to Start

Start with the WelcomeMessage.V1 structure definition in mls.proto to understand the field changes from message_cursor to welcome_metadata.


Macroscope summarized 77ac713.

@codabrink codabrink requested a review from a team as a code owner June 25, 2025 18:54
@codabrink codabrink changed the title add welcome metadata Add Welcome Metadata Jun 25, 2025
@nplasterer
Copy link
Contributor

Will this cause issues since the migration already ran for it?

@codabrink
Copy link
Contributor Author

It shouldn't, the migration that uses it will be dropped on the next deploy and it wasn't used anywhere. The Node go PR is on this branch and passes all tests.

@nplasterer nplasterer merged commit 9f01085 into main Jun 30, 2025
6 checks passed
@nplasterer nplasterer deleted the coda/welcome-metadata branch June 30, 2025 22:22
@github-actions
Copy link

github-actions bot commented Jul 7, 2025

πŸŽ‰ This PR is included in version 3.84.0 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants