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

Signing messages from runners #30

Closed
nikolasschoppmeier opened this issue Mar 25, 2022 · 1 comment
Closed

Signing messages from runners #30

nikolasschoppmeier opened this issue Mar 25, 2022 · 1 comment
Assignees
Milestone

Comments

@nikolasschoppmeier
Copy link

nikolasschoppmeier commented Mar 25, 2022

Background

There are a couple of usecases we need to solve for message signing from the runners perspective:

  • Messages broadcasted to other chains (in this case Terra), need to be signed. We need a way to sign a CosmWasmExecute message from the runner’s perspective.
  • We also need to sign messages in the CosmWasmExecuteMsg format that we are signing to get 2/3 of signatures back.
  • And also, we need to sign the message to update the validator set to get a network concensus.

Done when

  • We can sign any arbitrary sdk.Msg and also verify signatures
  • We allow user to load keyring path in which they have their terra keys stored which is going to be used for message signing.
@Vizualni
Copy link
Contributor

We can sign any arbitrary sdk.Msg and also verify signatures
Done as seen by this test:
https://github.com/VolumeFi/cronchain/blob/ca5cdc0f04592d96735ac979f21d9797de4b6c94/runner/client/sign_test.go#L66-L68

We allow user to load keyring path in which they have their terra keys stored which is going to be used for message signing.
I had to create this Init in our repo instead in the lens repo because lens repo is hardcoding to something that does not work with existing keyrings.

Config is set here
https://github.com/VolumeFi/cronchain/blob/ca5cdc0f04592d96735ac979f21d9797de4b6c94/runner/client/client.go#L18

and a ticket to create actual configs is here #48

@Vizualni Vizualni added this to the V 1.0 milestone Apr 12, 2022
@nikolasschoppmeier nikolasschoppmeier changed the title Signing messages from runners 1 Apr 25, 2022
@Vizualni Vizualni changed the title 1 Signing messages from runners Apr 26, 2022
@taariq taariq changed the title Signing messages from runners 1 Apr 26, 2022
@nikolasschoppmeier nikolasschoppmeier changed the title 1 Signing messages from runners Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

2 participants