Unlock and lock wallet when importmulti #337

Closed
opened 2022-03-31 18:36:11 +00:00 by vampik · 9 comments
vampik commented 2022-03-31 18:36:11 +00:00 (Migrated from gitlab.com)

importmulti is able to import private keys, and therefore needs unlocked wallet to store the private keys

We need to:

  1. unlock wallet
  2. importmulti
  3. lock wallet
importmulti is able to import private keys, and therefore needs unlocked wallet to store the private keys We need to: 1) unlock wallet 2) importmulti 3) lock wallet
vampik commented 2022-03-31 18:36:11 +00:00 (Migrated from gitlab.com)

assigned to @vampik

assigned to @vampik
vampik commented 2022-03-31 19:04:32 +00:00 (Migrated from gitlab.com)

mentioned in commit 28f44b093b

mentioned in commit 28f44b093b3939747adefa5c65cc5895da37eaad
vampik commented 2022-03-31 19:04:32 +00:00 (Migrated from gitlab.com)

mentioned in commit cc7f7fc103

mentioned in commit cc7f7fc1033cfa033a29eb1f1f99dbc6fe2d29bf
vampik commented 2022-03-31 19:06:22 +00:00 (Migrated from gitlab.com)

mentioned in merge request !93

mentioned in merge request !93
vampik commented 2022-03-31 19:09:50 +00:00 (Migrated from gitlab.com)

See ticket https://gitlab.com/PBSA/peerplays/-/issues/325

Now we don't need to unlock wallet to importmulti

See ticket https://gitlab.com/PBSA/peerplays/-/issues/325 Now we don't need to unlock wallet to `importmulti`
vampik commented 2022-04-01 12:20:59 +00:00 (Migrated from gitlab.com)

mentioned in commit ad0b5afb79

mentioned in commit ad0b5afb796c5b5f46585ec7d43359c109281fc4
serkixenos commented 2022-04-01 12:20:59 +00:00 (Migrated from gitlab.com)

mentioned in commit 4809619892

mentioned in commit 480961989240557bc645383fb3d4c820ec676353
serkixenos commented 2022-04-01 12:26:19 +00:00 (Migrated from gitlab.com)

assigned to @prandnum

assigned to @prandnum
prandnum commented 2022-04-04 18:36:39 +00:00 (Migrated from gitlab.com)

The following 6 scenarios are tested With v21:

  1. BTC withdraw
  2. BTC deposit
  3. active SON account change(by stopping active witnesses containers)

With v22:

  1. BTC withdraw
  2. BTC deposit
  3. active SON account change(by stopping active witnesses containers)
The following 6 scenarios are tested With v21: 1. BTC withdraw 1. BTC deposit 1. active SON account change(by stopping active witnesses containers) With v22: 1. BTC withdraw 1. BTC deposit 1. active SON account change(by stopping active witnesses containers)
prandnum (Migrated from gitlab.com) closed this issue 2022-04-04 18:36:40 +00:00
Sign in to join this conversation.
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: Peerplays_Blockchain/peerplays_migrated#337
No description provided.