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

Add BLS signature support for consensus #159

Open
naitik-supraoracles opened this issue Jan 2, 2025 · 0 comments
Open

Add BLS signature support for consensus #159

naitik-supraoracles opened this issue Jan 2, 2025 · 0 comments

Comments

@naitik-supraoracles
Copy link

Description

This is proposal for adding support of BLS sig in consensus process. Below are the potential primary changes which will be required in order to support bls signatures.

Overview of required changes

  • In vm-genesis/lib.rs, we have this Validator struct which is being used in smr-moonshot , here we have to add field for having BLS publickey.

  • here we have this ed15519 module which wrapper over original type with some implemented required traits to support over other aptos-core structs. so that we also have to implement new module with wrapper struct on blsttc as per aptos-core required traits.

  • in validator_config.rs we have this ValidatorConfig where we need to add field for BLS publickey. it will also require some other changes accordingly. This change also requires changes in move layer, because it will be going to deserialise there, also requires other changes in move layer.

  • We can also expect many changes on move side with many structs and methods to support new changes for BLS signatures.

  • Other small changes need to be done while implementing to tackle if any other issues may arise.

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

No branches or pull requests

1 participant