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

solana-keygen doesn't generate keys from keys (BIP32 support) #5246

Closed
garious opened this issue Jul 23, 2019 · 8 comments
Closed

solana-keygen doesn't generate keys from keys (BIP32 support) #5246

garious opened this issue Jul 23, 2019 · 8 comments
Assignees
Milestone

Comments

@garious
Copy link
Contributor

garious commented Jul 23, 2019

Problem

A user may prefer to only hold one private key and want to generate more private keys deterministically from that first one (typically called an HD wallet).

Proposed Solution

Add a bip32 implementation to solana-keygen. Consider if it should be in solana-wallet as well.

https://github.com/bitcoin/bips/blob/master/bip-0032.mediawiki

Some options:

@fkbenjamin
Copy link

fkbenjamin commented Aug 9, 2019

Just a short warning that the combination of bip32 and ed25519 can be vulnerable to certain attacks. Details can be found here and here.

I'm not an expert on the topic, but maybe something like this is worth a look: https://github.com/w3f/hd-ed25519

@garious
Copy link
Contributor Author

garious commented Aug 9, 2019

@fkbenjamin, thank you so much!!

@mvines mvines changed the title solana-keygen doesn't generate keys from keys solana-keygen doesn't generate keys from keys (BIP32 support) Sep 12, 2019
@garious garious mentioned this issue Oct 10, 2019
@jstarry jstarry self-assigned this Oct 31, 2019
@stale
Copy link

stale bot commented Oct 31, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the stale [bot only] Added to stale content; results in auto-close after a week. label Oct 31, 2020
@CriesofCarrots
Copy link
Contributor

Un-staling. This would actually be nice to address sometime soon-ish. With more wallet adoption, we're getting a lot of support questions about how to derive keypairs that match those of other wallets.

@stale stale bot removed the stale [bot only] Added to stale content; results in auto-close after a week. label Oct 31, 2020
@CriesofCarrots CriesofCarrots self-assigned this Oct 31, 2020
@t-nelson
Copy link
Contributor

t-nelson commented Nov 2, 2020

I've got an eye on https://github.com/jpopesculian/ed25519-dalek-bip32. Unfortunately it's not quite flexible enough for our usage in its current form. I'll try to get some issues up to their repos this week

@romeo4934
Copy link

any update on this issue?

@CriesofCarrots
Copy link
Contributor

CriesofCarrots commented Apr 19, 2021

any update on this issue?

Hi @antoineherzog , just an fyi that we're actively working on this now! Keep an eye on this issue for updates.

@CriesofCarrots
Copy link
Contributor

I'm going to consider this issue completed by #16942 -- bip32 HD from seed phrase and pass phrase is now supported in solana command-line tools!
There is still additional work needed to support bip32 derivation from Keypair-file inputs; will open a separate issue describing.

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

No branches or pull requests

6 participants