-
Notifications
You must be signed in to change notification settings - Fork 96
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
Allow to set custom derivation path in "Sign in" #3759
Comments
Thanks for opening your first issue here! Be sure to follow the issue template for bug report or feature request! |
Note: We need to make sure we add a warning sign to let users know about the risk involved. |
In case, this is what you need to implement this: https://github.com/alepop/lisk-hd-key |
I just want to add some thoughts about this. This could be the default way to access lisk desktop (and still keep the passphrase as it is for backward-compatibility, but as second option) and not only a tool to access ledger and trezor accounts. For me like electrum is ideal:
If user forget the password, they can always re-import the seed from the backup and set a new password. In this way you avoid lisk users to manage constantly the passphrase (aka the private key) every time they need to access the wallet. This could be a very nice improvement. |
@hirishh thank you for the suggestion, we discussed the exact same idea internally with our research yesterday. We have a roadmap objective for secure signing as part of that research we are doing this improvement. In coming months you will hear about the approach once research team publishes on the forum, thank you once again for all the input, it will be valuable during our research 🙏🏼 |
I have another hint for you on how it could be implemented and create less/no confusion for lisk users. First of all I have a question: is the current lisk passphrase compatible with bip32 alghoritm/checksum? If yes:
In this way everything is completely transparent to the user and they do not need to choose between hd-wallet (= Hierarchical Deterministic Wallets) or legacy lisk passphrase method. |
Hi @hirish with regards to Lisk Desktop supporting recovery phrase for 2.1.0 release (#3759) I wanted to give you an update. We have decided the following approach of implementation for 2.1.0.
We will provide a warning message to ensure the users are aware of security threats and mention the purpose of this feature(to recover funds from a hardware wallet in case of emergency, not for everyday usage). The recovery phrase approach will be removed with Lisk Desktop 2.2.0 release. |
The blocker should be resolved with LiskArchive/lisk-sdk#6816 |
…on-path Allow to set custom derivation path - Closes #3759
Description
A quick fix for allowing hardware wallets users to access their funds is to give the ability to choose a custom derivation path during "Sign in with passphrase"; in this way, even if hardware wallets are not supported yet, users can access their funds using recovery passphrase.
Motivation
To give a quick and immediate solution for hardware wallet users.
The text was updated successfully, but these errors were encountered: