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

Allow unconfirmed members. Fixes #205 #208

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

ekr
Copy link
Collaborator

@ekr ekr commented Oct 21, 2023

Change the definition of member to include only having been added to the group and not contributed to the group and explains what that might mean.

Note. this will need WG consensus to change.

Change the definition of member to include only having been added
to the group and not contributed to the group and explains what
that might mean.
secret in a manner verifiable by other members of the group, other members
cannot assume that the client is a member of the group.
cannot assume that the client is a member of the group; for instance, the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While you're here, we might want to add that while other clients in the group shouldn't assume the new client is a Member as per the definition above, they should assume that that client can indeed decrypt all messages that they send after it was added to the group.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the suggestion. I actually think there are two different things one might assume:

(1) that the client might have access to the messages, so you have no secrets from them
(2) that the client will have access to the messages, so they will (for instance) act on them.

I think you should assume (1) but not (2).

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, I think that's the distinction we want here.

@beurdouche beurdouche merged commit 3871650 into mlswg:main Nov 7, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants