-
-
Notifications
You must be signed in to change notification settings - Fork 179
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
get_child() has nothing to do with BIP32_PATH #14
Comments
Not sure if you ever figured this out, but I'm also seeing inconsistencies. Trying all kinds of things but can't ever seem to get it to match addresses from https://iancoleman.io/bip39/ or the pycoin library I thought about swapping this out with. Tried some things like
but not matching up at all. |
Same here |
Could this be releated too: Bug NoticeVersions of bitmerchant prior to 0.1.8 contained a caching bug that may have resulted in calls to bip32.Wallet.get_child to return incorrect results. All affected versions were removed from pypi, and no users are known to have been affected by this bug. The steps to reproduce the bug are unlikely and do not match the typical usage patterns of bitmerchant. At this time, no users are known to have been affected by this bug. If you have been affected by this bug and need help recovering any lost or misplaced coins, please contact me directly at steven.buss+bitmerchant@gmail.com. The affected versions of bitmerchant have been removed from pypi. They have not been untagged in git. The two possible failure scenarios are: misplaced coins and stolen coins https://github.com/sbuss/bitmerchant/blob/master/BUG_NOTICE.rst |
It seems that the chain codes used for calculating the child keys are completely wrong - it does HMAC on the mnemonic words themselves instead of the 128-bit seed entropy of the mnemonic. See Just put this code at the beginning of from_master_key to fix the bug:
|
Hi,
I read your code, and found that the function get_child() has nothing to do with BIP32_PATH, this might be an issue.
https://github.com/ranaroussi/pywallet/blob/master/pywallet/wallet.py#L158
https://github.com/ranaroussi/pywallet/blob/master/pywallet/utils/bip32.py#L190
the address generated doesn't match the bip32_path
The text was updated successfully, but these errors were encountered: