remove coinstring, replace with bs58check, update vulnerable dependencies

cryptocoinjs / hdkey

JavaScript component for Bitcoin hierarchical deterministic keys (BIP32)

MIT License

201 stars 74 forks source link

remove coinstring, replace with bs58check, update vulnerable dependencies #30

Closed nitowa closed 5 years ago

nitowa commented 5 years ago

I implemented the bs59check drop-in replacement for coinstring as discussed in https://github.com/cryptocoinjs/hdkey/pull/29

I also updated all vulnerable dependencies. All tests pass and browser-test coveralls at least behave identically although I don't have the environment to try them out completely.

jprichardson commented 5 years ago

I'm okay with this. @kklash please take a look.