Skip to content
This repository was archived by the owner on Apr 3, 2019. It is now read-only.

Conversation

@gabegattis
Copy link
Contributor

@gabegattis gabegattis commented Dec 7, 2016

#96

This was referenced Dec 7, 2016
@matiu
Copy link
Contributor

matiu commented Dec 8, 2016

What is the motivation for this? Key derivation is VERY slow in JS.

@braydonf
Copy link
Contributor

braydonf commented Dec 8, 2016

Regarding the speed of derivation, FWIW: https://github.com/cryptocoinjs/hdkey implements derivivaiton using secp256k1 C bindings which is much faster, and bcoin has also started to do something similar at https://github.com/bcoin-org/bcoin/tree/master/lib/hd though is still using cache.

@matiu
Copy link
Contributor

matiu commented Dec 8, 2016 via email

@fanatid
Copy link
Contributor

fanatid commented Dec 8, 2016

@matiu secp256k1 uses elliptic in browser

@gabegattis
Copy link
Contributor Author

@matiu How often are we actually re-deriving a key that we previously derived? It seems like that would be a very rare situation. We don't reuse addresses in Copay, so it seems like we would normally just derive a key once and never need it again.

@kleetus
Copy link
Contributor

kleetus commented Feb 15, 2017

@gabegattis based on conversations, both in this thread and external, is this code still ready to merge?

@isocolsky
Copy link
Contributor

ACK

1 similar comment
@kleetus
Copy link
Contributor

kleetus commented Mar 1, 2017

ACK

@kleetus kleetus merged commit ed02685 into bitpay:master Mar 1, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants