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

Archive key material on wallet closure #3814

Merged
merged 17 commits into from
Jun 11, 2024
Merged

Conversation

tomaszslabon
Copy link
Contributor

@tomaszslabon tomaszslabon commented May 16, 2024

#Refs: #3797.
This PR handles archiving key material on wallet closures.
Archiving the wallet data happens in two situations:

  • when the WalletClosed event is received
  • on node start, if there are closed wallets in the node's wallet registry.

Archiving wallet's key material causes the wallet data to be moved to the archive directory and the wallet is removed
form the registry's cache..
Such wallets will no longer participate in the RFC-12 coordination.

@tomaszslabon tomaszslabon force-pushed the archive-key-material branch 2 times, most recently from 849019d to 855631d Compare June 7, 2024 08:26
@tomaszslabon tomaszslabon marked this pull request as ready for review June 10, 2024 07:14
pkg/tbtc/node.go Outdated Show resolved Hide resolved
pkg/tbtc/node.go Outdated Show resolved Hide resolved
pkg/tbtc/registry.go Outdated Show resolved Hide resolved
pkg/tbtc/node.go Outdated Show resolved Hide resolved
pkg/tbtc/registry.go Outdated Show resolved Hide resolved
pkg/tbtc/registry.go Show resolved Hide resolved
pkg/tbtc/node.go Show resolved Hide resolved
pkg/tbtc/node.go Outdated Show resolved Hide resolved
lukasz-zimnoch
lukasz-zimnoch previously approved these changes Jun 11, 2024
@lukasz-zimnoch lukasz-zimnoch merged commit efd3f20 into main Jun 11, 2024
29 checks passed
@lukasz-zimnoch lukasz-zimnoch deleted the archive-key-material branch June 11, 2024 13:55
@lukasz-zimnoch lukasz-zimnoch added this to the v2.1.0 milestone Jun 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants