-
Notifications
You must be signed in to change notification settings - Fork 68
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
IF: updated qc_chain to use bls_public_key as finalizer id, completed internal wiring for signing + verification #1616
IF: updated qc_chain to use bls_public_key as finalizer id, completed internal wiring for signing + verification #1616
Conversation
…ernal wiring for signing and signature verification
Co-authored-by: Kevin Heifner <[email protected]>
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
…ded finalizer_count parameter to quorum_certificate constructor, and resize hs_bitset to the expected size
…of the active finalizer set. Improved test to reflect expected operational conditions better
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
@@ -3,7 +3,7 @@ | |||
#include <eosio/hotstuff/base_pacemaker.hpp> | |||
#include <eosio/hotstuff/qc_chain.hpp> | |||
|
|||
#include <eosio/chain/finalizer_set.hpp> | |||
//#include <eosio/chain/finalizer_set.hpp> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be removed. But finalizer_set
needs to be forward declared.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I did remove the commented out line, but I'm not sure if/how the forward declaration would be used, since finalizer_set is included in the base_pacemaker.hpp header.
static finalizer_set create_fs(std::vector<std::string> keys){ | ||
|
||
std::vector<fc::crypto::blslib::bls_private_key> sks; | ||
//std::vector<fc::crypto::blslib::bls_public_key> pks; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove
Co-authored-by: Gregory Popovitch <[email protected]>
Co-authored-by: Gregory Popovitch <[email protected]>
ac6400a
to
4ffc264
Compare
Note:start |
Resolves #1610 and #1626.