navcoin-core icon indicating copy to clipboard operation
navcoin-core copied to clipboard

Published 20 hours ago • verified publisher icon navcoin

Derivation path

Open aguycalled opened this issue 5 years ago • 13 comments

Currently navcoin core shares with bitcoin the derivation path used for the generation of new addresses (m/0'/0'), this should be changed to m/44'/130' while preserving backwards compatibility with old wallets.

aguycalled avatar May 22 '19 20:05 aguycalled

@aguycalled we should add an issue over at NavPay as well, and Next?

We only have 3 wallets to update right?

NEXT uses core? So do we have anything to change on that end?

mxaddict avatar May 22 '19 20:05 mxaddict

NEXT uses core, NavPay should see an issue opened too

aguycalled avatar May 22 '19 20:05 aguycalled

NEXT uses core, NavPay should see an issue opened too

ACK

mxaddict avatar May 22 '19 20:05 mxaddict

I think NavPay is easier as it's just changing the default on creation and the default on import

Then adding a note on import page.

mxaddict avatar May 22 '19 20:05 mxaddict

I think NavPay is easier as it's just changing the default on creation and the default on import

Then adding a note on import page.

This statement was WRONG :+1:

mxaddict avatar May 23 '19 20:05 mxaddict

i do not see anything on this here: https://github.com/bitcoin/bips/blob/master/bip-0130.mediawiki
so you're talkin about m/44'/130'

alexeyneu avatar May 31 '19 18:05 alexeyneu

and i found this: https://github.com/navcoin/navcoin-core/blob/498e75f2b915bf5d8fccc10c1080ed68b478fceb/src/wallet/wallet.cpp#L119
so what is it you finally wanna?

alexeyneu avatar May 31 '19 22:05 alexeyneu

you are right, updated issue description

aguycalled avatar Jun 01 '19 08:06 aguycalled

Currently navcoin core shares with bitcoin the derivation path used for the generation of new addresses (m/0'/0'), this should be changed to m/44'/130' while preserving backwards compatibility with old wallets.

it'll be 44'/130'/0'/0'/0'
in 44'/130'/0' all three lose its meanings

EDIT:
you can go with letter stuff

alexeyneu avatar Jun 02 '19 20:06 alexeyneu

@alexeyneu What do you mean by "letter stuff"

mxaddict avatar Jun 03 '19 16:06 mxaddict

@alexeyneu are you working on this? If so I can assign this issue to you.

mxaddict avatar Jun 13 '19 14:06 mxaddict

may be i'll do it , may be not the bad is that you'd to add mnemonic early . And now all this deriv path stuff goes straight to user coz no one knows what he'd used to export it.
say in mew during import you choose path used , but before that you did it on export . So how i see it is to add option to importmnemonic like -before17 .
on the other hand it's new stuff so is not being used by many.

alexeyneu avatar Jun 14 '19 00:06 alexeyneu

Additional info: https://github.com/bitcoin/bips/blob/master/bip-0044.mediawiki https://github.com/satoshilabs/slips/blob/master/slip-0044.md

mxaddict avatar Jun 18 '19 15:06 mxaddict