maidsafe / bls_dkg

Implementation of a BLS DKG mechanism, requires signing key, encryption key and SocketAddr of participants
BSD 3-Clause "New" or "Revised" License
22 stars 17 forks source link

chore: bump blsttc from 3.1.0 to 5.1.0 #121

Closed dan-da closed 2 years ago

dan-da commented 2 years ago

We also:

These make us compatible with deps in blsttc.


tests are passing locally.

running 6 tests
test key_gen::tests::having_min_unresponsive_nodes_cause_block ... ok
test key_gen::tests::all_nodes_being_responsive ... ok
test key_gen::tests::threshold_encrypt ... ok
test key_gen::tests::threshold_signature ... ok
test key_gen::tests::having_max_unresponsive_nodes_still_work ... ok
test key_gen::tests::network_churning has been running for over 60 seconds
test key_gen::tests::network_churning ... ok

test result: ok. 6 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 224.22s

   Doc-tests bls_dkg

running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.00s
davidrusu commented 2 years ago

@iancoleman I think you had mentioned that you had done this update to bls_dkg in one of your branches? Want to look this over and see if it's equivalent?

iancoleman commented 2 years ago

This matches my local changes to bls_dkg (not exactly but in effect they are the same) so LGTM.

I didn't import ff::Field and changed sk_val = Fr::default()

I don't have .ok()? on poly.interpolate L980 of mod.rs

dan-da commented 2 years ago

i will re-work this a bit to accomodate changes in #123.