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

Add rekey feature with blank key support #3125

Merged
merged 5 commits into from
Jul 29, 2024
Merged

Conversation

jamshale
Copy link
Contributor

@jamshale jamshale commented Jul 25, 2024

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.

Signed-off-by: jamshale <jamiehalebc@gmail.com>
@jamshale jamshale marked this pull request as ready for review July 25, 2024 18:41
@jamshale jamshale requested a review from ianco July 25, 2024 20:43
@jamshale
Copy link
Contributor Author

rekey derivation method support has been added.

esune
esune previously approved these changes Jul 26, 2024
Copy link
Contributor

@esune esune left a 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>
Copy link
Member

@ianco ianco left a 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

@jamshale jamshale enabled auto-merge (squash) July 29, 2024 17:17
@jamshale jamshale merged commit 77ca06d into hyperledger:main Jul 29, 2024
7 checks passed
Copy link

sonarcloud bot commented Jul 29, 2024

darshilnb pushed a commit to Northern-Block/aries-cloudagent-python that referenced this pull request Sep 5, 2024
* 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>
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.

4 participants