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

Austin retreat #99

Merged
merged 81 commits into from
Mar 1, 2022
Merged

Austin retreat #99

merged 81 commits into from
Mar 1, 2022

Conversation

JesseAbram
Copy link
Member

No description provided.

thor314 and others added 30 commits January 31, 2022 13:06
- change module common to package module
- add input to register()
- change account_registration() to register()
- add OCWMessage to crypto/common and use that in the noes
- relayer::post() sends only 1 message, not messages
* setup clap CLI

* keygen async setup from Clap

* keygen async

* typechecks pass

* yoink, structopt

* reorg subdirs

* sm-manager

* keygen + signing - compilation errors

* pending keygen ownership bug

* async ownership bugfix
@JesseAbram JesseAbram requested review from thor314 and davfra February 28, 2022 17:10
crypto/protocol/src/node.rs Outdated Show resolved Hide resolved
crypto/protocol/src/user.rs Show resolved Hide resolved
crypto/protocol/src/user.rs Show resolved Hide resolved
crypto/signing-client/src/com_manager.rs Outdated Show resolved Hide resolved
crypto/signing-client/src/sign.rs Outdated Show resolved Hide resolved
crypto/signing-client/src/sign.rs Outdated Show resolved Hide resolved
pallets/propagation/src/lib.rs Show resolved Hide resolved
pallets/relayer/src/lib.rs Outdated Show resolved Hide resolved
pallets/relayer/src/lib.rs Show resolved Hide resolved
pallets/relayer/src/lib.rs Outdated Show resolved Hide resolved
thor314
thor314 previously approved these changes Mar 1, 2022
davfra added 2 commits March 1, 2022 11:27
- Design change: The communication manager is now always running instead of being called when necessary.
davfra
davfra previously requested changes Mar 1, 2022
Copy link
Contributor

@davfra davfra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

following README.md section 'testnet'... ./target/release/test_register throws an error for me.
I think we should relolve this before merging.

@JesseAbram JesseAbram marked this pull request as ready for review March 1, 2022 13:52
@JesseAbram JesseAbram merged commit 2bc2f17 into master Mar 1, 2022
@JesseAbram JesseAbram deleted the crypto branch March 1, 2022 19:04
fjarri pushed a commit that referenced this pull request May 5, 2023
* init protocol repo commit

* block: Zeroize compile bug

* resolve zeroize-derive dependency issue

* skeleton: keygen and signing

* WIP: add extrinsic to user::send_tx()

* add comments

* ' add comment'

* update subxt

* add common package

* change relayer from lindell17 to common

* add conditional std attribute to common

* close #68

- change module common to package module

* add send_registration(), struct SigRequest

* close #72

* Delete etp-lindell17 directory

* WIP: testing extrinsics

* pallet relayer: unify account_registration and register()
- add input to register()
- change account_registration() to register()

* add testing-user-clients to call extrincics in CLI

* mess with node to signing-client communication

- add OCWMessage to crypto/common and use that in the noes
- relayer::post() sends only 1 message, not messages

* Crypto protospec (#84)

* setup clap CLI

* keygen async setup from Clap

* keygen async

* typechecks pass

* yoink, structopt

* reorg subdirs

* sm-manager

* keygen + signing - compilation errors

* pending keygen ownership bug

* async ownership bugfix

* fix signing-client decoding

* minor changes

* fix last merge: refactor protocol::user

* move common.rs into its own package

* add SigResponse to relayer::events::TransactionPropagated

* store json key (#88)

* store json key

* refactor

* to do fix

* add wait_for_finalized_success() to request_sig_gen()

- when receiving a result, result.find_first_event requires wait_for_finalized_success()
- request_sig_gen() returns SigResponse

* Crypto protospec (#89)

* setup clap CLI

* keygen async setup from Clap

* keygen async

* typechecks pass

* yoink, structopt

* reorg subdirs

* sm-manager

* keygen + signing - compilation errors

* pending keygen ownership bug

* async ownership bugfix

* blocking: handling error on 6 of 7 keygen

* keygen bug documented

* Store keys (#90)

* store json key

* refactor

* to do fix

* alice send

* add sign_message() to User

* cli separated

* cli separated (#92)

* add fn sign_message() and a test to run it

* integration

* add scripts to run 2 signing-clients locally

* work on registration

* fix scripts/alice.sh and scripts/bob.sh

* add send() to registration

* remove main.rs in alice-send

* unbreak wrap cli

* registration complete

* user await sign()

* add gg20-sm-manager's routines to signing-client

* fix sm-manager in signing-client

* add println!() for debugging

* copy local-share1.json to root

- this key needs to be in the root for the current testnet setup
- the other key local-share2.json is copied to root in the User registration

* change Rocket.toml settings

* - add logs

* adjust signing-client for node1

* change order of SignCli::sign_cli.parties

- changed from vec![1,2] to vec![2,1]
- THIS made the signature generation possible. WHY?
- change for user and signing-node

* add testnet instructions to README.md

* update README.md

* comment changes

* tests

* fmt

* todo

* event docs

* pipeline

* update README.md

* remove start_com_manager()

- Design change: The communication manager is now always running instead of being called when necessary.

* remove unused code

* build fix

* fix non compilation

* fmt

* build fix

Co-authored-by: Thor <thorck@protonmail.com>
Co-authored-by: davfra <64629389davfra@users.noreply.github.com>
Co-authored-by: David <818daf@gmail.com>
Co-authored-by: Thor <7041313+thor314@users.noreply.github.com>
Co-authored-by: davfra <64629389+davfra@users.noreply.github.com>
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

Successfully merging this pull request may close these issues.

3 participants