Skip to content

Keystore compatibility with cli - #898

Merged
ashchan merged 10 commits into
developfrom
keystore-compatibility-with-cli
Aug 29, 2019
Merged

Keystore compatibility with cli#898
ashchan merged 10 commits into
developfrom
keystore-compatibility-with-cli

Conversation

@ashchan

@ashchan ashchan commented Aug 28, 2019

Copy link
Copy Markdown
Contributor

Having two issues needing fix:

  • Neuron doesn't seem to be able to handle CKB cli's standard keystore( 262144 of n param), resulting OPENSSL MEMORY_LIMIT_EXCEEDED error.
  • Neuron keystore password logic has flaw.

Worth noting

We don't aim 100% compatibility. CKB CLI uses the master private key to derive mainnet/testnet addresses. Neuron on the other hand generates HD addresses (both receiving and change) with full path (m/44'/309'/0'/0/n, and m/44'/309'/0'/1/n).

This means the address you see from cli will NOT appear in Neuron.

One more thing, our Keystore format doesn't conform to Ethereum wallets in the way that we save the encrypted extended master key, not the encrypted master private key.

To fix

  • CLI Standard keystore import
  • password/mac algorithm

@ashchan ashchan self-assigned this Aug 28, 2019
@ashchan
ashchan force-pushed the keystore-compatibility-with-cli branch from 1213f88 to 7ea91f0 Compare August 28, 2019 11:32
@ashchan
ashchan marked this pull request as ready for review August 28, 2019 13:24
@ashchan
ashchan merged commit d057490 into develop Aug 29, 2019
@ashchan
ashchan deleted the keystore-compatibility-with-cli branch August 29, 2019 08:28
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

Successfully merging this pull request may close these issues.

3 participants