-
Notifications
You must be signed in to change notification settings - Fork 513
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
Add rekey feature with blank key support #3125
Conversation
Signed-off-by: jamshale <jamiehalebc@gmail.com>
rekey derivation method support has been added. |
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 think this looks good 👍🏻
Signed-off-by: jamshale <jamiehalebc@gmail.com>
Signed-off-by: jamshale <jamiehalebc@gmail.com>
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.
Looks good to me
Quality Gate passedIssues Measures |
* Add rekey feature with blank key support Signed-off-by: jamshale <jamiehalebc@gmail.com> * Refactor / Remove other key exists check / unit tests Signed-off-by: jamshale <jamiehalebc@gmail.com> * Remove no wallet key test from store config Signed-off-by: jamshale <jamiehalebc@gmail.com> --------- Signed-off-by: jamshale <jamiehalebc@gmail.com> Co-authored-by: Ian Costanzo <ian@anon-solutions.ca>
See #2999.
This adds the support to change wallet keys and also allows any wallets created with a blank key before it was required in 0.12.0 to change their key and unblock the upgrade.
This allow an agent stuck in a state where they used a blank key to configure a wallet-key that won't be used, but will be replaced by the
rekey
variable. After using rekey they can then use that key in the wallet-key config.