feat: presign for validator account #6747
Conversation
|
|
There was a problem hiding this comment.
Thank you for the PR! I have tested it and the feature is working. I have left some comments.
The CI check on formatting failed, also it would be great if you can sign the CLA.
I think it will also be nice to have a note highlighting the feature in the documentation, maybe before this section: https://lighthouse-book.sigmaprime.io/voluntary-exit.html#initiating-a-voluntary-exit
Thanks!
| eprintln!( | ||
| "Publishing a voluntary exit for validator: {} \n", | ||
| keypair.pk | ||
|
|
| let string_output = serde_json::to_string_pretty(&signed_voluntary_exit) | ||
| .map_err(|e| format!("Unable to convert to JSON: {}", e))?; | ||
|
|
||
| println!("{}", string_output); |
There was a problem hiding this comment.
While working on #6612 , initially I also printed out the signature. But then I realise it probably isn't much useful in practice - if users want to pre-sign an exit, they probably want the signature saved to a file. So I modified in this commit in that PR:
5a4f928
We can hear from others about this and edit accordingly.
| @@ -74,6 +76,15 @@ pub fn cli_app() -> Command { | |||
| .action(ArgAction::SetTrue) | |||
| .help_heading(FLAG_HEADER) | |||
| ) | |||
| .arg( | |||
| Arg::new(PRESIGN) | |||
There was a problem hiding this comment.
I have changed the flag name in #6612 to --presign as per the comment: #6612 (comment)
So please ignore the above
Issue Addressed
#6746
Proposed Changes
Add a --presign flag to emit the json output to stdout instead of publishing the exit
Additional Info
The validator-manager is also a surface area to support via the http client interface