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.

Sign up for GitHub

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

Jump to bottom

Strongest credential type #225

Closed
ekr opened this issue Nov 8, 2023 · 6 comments
Closed

Strongest credential type #225

ekr opened this issue Nov 8, 2023 · 6 comments
Assignees
@beurdouche

Comments

@ekr
Copy link
Collaborator

ekr commented Nov 8, 2023

RECOMMENDATION: Select the strongest MLS Credential type available among
the target members of an MLS group.

What is this trying to say. Is it the strongest common type?
@ekr
Copy link
Collaborator Author

ekr commented Nov 9, 2023

@bifurcation What do you think this is saying?

@Bren2010
Copy link
Contributor

Don't use BasicCredential if everyone supports X509Credential. Don't use X509Credential if everyone supports _____ (fill in blank)

@rohan-wire
Copy link
Contributor

RECOMMENDATION: Select the strongest MLS Credential type available among
the target members of an MLS group.

What is this trying to say. Is it the strongest common type?

The wording already in the recommendation seems like the clearest description of "strongest common type".

@beurdouche
Copy link
Member

"strongest" what is the question... I'll clarify...

@beurdouche beurdouche self-assigned this Nov 10, 2023
@ekr
Copy link
Collaborator Author

ekr commented Nov 10, 2023

My concern wasn't strongest but "available"

@ekr
Copy link
Collaborator Author

ekr commented Nov 10, 2023

See: #235

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

@beurdouche beurdouche

Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@Bren2010 @ekr @beurdouche @rohan-wire