Skip to content

Conversation

@instagibbs
Copy link
Contributor

resolves #547

Previous API was improperly overloading the specific blinding key calls, and incorrectly on top of that.

Instead implement importmasterblindingkey and dumpmasterblindingkey and check that dumpwallet details are enough to recover same blinded wallet addresses.

@instagibbs instagibbs added bug and removed bug labels Mar 29, 2019
@instagibbs instagibbs requested a review from stevenroose March 29, 2019 17:09
@instagibbs instagibbs added the needs port Needs backport to a different branch label Mar 29, 2019
@instagibbs instagibbs force-pushed the test_blind_recovery branch from c6d4bad to 9acbd24 Compare March 29, 2019 17:20
@stevenroose stevenroose merged commit 9acbd24 into ElementsProject:master Apr 1, 2019
stevenroose added a commit that referenced this pull request Apr 1, 2019
9acbd24 Make new actually working master blinding calls, test (Gregory Sanders)

Pull request description:

  resolves #547

  Previous API was improperly overloading the specific blinding key calls, and incorrectly on top of that.

  Instead implement `importmasterblindingkey` and `dumpmasterblindingkey` and check that `dumpwallet` details are enough to recover same blinded wallet addresses.

Tree-SHA512: d9d954d1cb787182d99b464cc6a80e8383d3ea71e432e6fd0e244b5f5a460b9adfcdc14e38fe841c5a5e2fef1345da78217d519a23a804d5066998828103efd1
@stevenroose
Copy link
Contributor

tACK 9acbd24

@instagibbs instagibbs removed the needs port Needs backport to a different branch label Jun 5, 2019
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.

add test for blinding master key backup/restore

2 participants