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

The restriction on member encryption keys (RSA only) should be checked as early as possible and made clear in the doc #5920

Open
achamayou opened this issue Jan 11, 2024 · 0 comments
Labels

Comments

@achamayou
Copy link
Member

This restriction is currently raised when a member first tries to active, which is early enough to avoid impact to the consensus, but not user friendly.

Instead, CCF should check (and raise an error if needed) in two places:

  1. In the start config when members are provided there
  2. In the validate() call for set_member

In addition, the doc should make clear why EC keys are not supported for this purpose.

@achamayou achamayou added the TODO label Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant