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

Retrieve BLS Public Key #215

Closed
dutterbutter opened this issue Feb 5, 2020 · 1 comment · Fixed by #371
Closed

Retrieve BLS Public Key #215

dutterbutter opened this issue Feb 5, 2020 · 1 comment · Fixed by #371
Labels
Priority: 4 - Low Limited impact and can be implemented at any time

Comments

@dutterbutter
Copy link
Contributor

dutterbutter commented Feb 5, 2020

Task summary

Need the ability to retrieve the public key from a BLS signature, this functionality is needed to verify the signature of messages.

Required for the check block messages part of block validation, specifically for bls messages as the public keys are a required param for the verify_bls_aggregate function.

Other information and links

In our codebase, this would be used during block validation process in the Chainsyncer component here.

@dutterbutter dutterbutter added Status: Needs Triage Issue has unresolved discussions and/or needs to be assigned a priority and assignee Crypto Priority: 4 - Low Limited impact and can be implemented at any time and removed Status: Needs Triage Issue has unresolved discussions and/or needs to be assigned a priority and assignee labels Feb 5, 2020
@austinabell
Copy link
Contributor

austinabell commented Feb 5, 2020

You mean public key from Address right? I think it may be unlikely to be needed since it's very simple to check if an address is BLS and get the payload (public key)

Edit: nvm it probably will be needed to pull a key address from an actor address

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority: 4 - Low Limited impact and can be implemented at any time
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants