Replies: 1 comment 2 replies
-
|
Nice. Yes, important to simplify across. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
There are a couple of open issues requesting wallet password prompts to be put in for Burn BSQ, Asset fee listing, Bonded Reputation, Unlock Bond, Make Proposal, Vote, and View Private Keys. There is already a wallet password prompt at app startup, and when sending from BTC & BSQ wallets. A previous PR was cited as an example of how to do it. But actually a more consistent way to unlock the wallet would be better.
The model used in other wallets like
monero-wallet-cliand also Unixsudois, upon a privileged action, to authenticate and keep that authentication for a certain period of time, e.g. 1 minute. That way if you are doing repeated wallet actions you don't have to enter the password every time. The wallet automatically locks itself after a minute of inactivity.The other advantage is that you intercept the privilege escalation at the point it is used rather than spreading the same code all throughout the GUI. This makes code cleaner, more robust, and devs don't have to remember to code a password prompt every time a new wallet feature is added (like the one recently added to the Multisig Payout Tool).
We already have some form of unlock and lock on timeout implemented in
CoreWalletsService.java. Perhaps this can be utilized. I just need to figure out the details, and wanted to communicate that I'm working on it as an overall solution to #5152 and #5276cc: @bisq-network/bisq-devs @pazza83
Beta Was this translation helpful? Give feedback.
All reactions