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

Use BIP44 Compliant Tree Generation #70

Closed
danfinlay opened this issue Mar 24, 2016 · 1 comment
Closed

Use BIP44 Compliant Tree Generation #70

danfinlay opened this issue Mar 24, 2016 · 1 comment

Comments

@danfinlay
Copy link
Contributor

This will allow Metamask to share accounts with testrpc, as described in this discussion (Thanks @tcoulter!)

Will also allow Metamask to share accounts with Jaxx, a current wallet that is fairly popular.

@danfinlay
Copy link
Contributor Author

Note from conversation with Chris Lundkvist:

danfinlay [3:45 PM]

Hey Chris, I’m trying to get Metamask BIP44 compliant so it interoperates with testrpc, and since we use light-wallet I thought you’d be the guy to ask. Our current wallet generation is this: https://github.com/MetaMask/metamask-plugin/blob/master/app/scripts/lib/idStore.js#L269-L276
I’m not really clear what to do differently to make it BIP44

chrislundkvist [4:10 PM]

There is a function setDefaultHDPath where you can set the path. So you would call this function before you do anything else. I think what you want as your path is the string m/44'/60'/0'/0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant