Skip to content
This repository has been archived by the owner on Dec 11, 2019. It is now read-only.

Digest method not supported when making contribution #7875

Closed
bsclifton opened this issue Mar 24, 2017 · 3 comments
Closed

Digest method not supported when making contribution #7875

bsclifton opened this issue Mar 24, 2017 · 3 comments

Comments

@bsclifton
Copy link
Member

bsclifton commented Mar 24, 2017

Test plan

  1. Update your ledger state file so that a contribution will be processed soon
  2. Start Brave via the command line
  3. Wait until the contribution is process
  4. Console (from the command line) should not output any errors / warnings related to hash digest

Original issue description

Reported by @mrose17

When running 0.14.0 Preview 5 from the command line, we're seeing the following when trying to make a contribution:

callback: Error: Digest method not supported
Error: Digest method not supported
    at new Hash (crypto.js:56:18)
    at Hash (crypto.js:55:12)
    at ripemd160 (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitcoinjs-lib/src/crypto.js:12:10)
    at Object.hash160 (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitcoinjs-lib/src/crypto.js:4:10)
    at HDNode.getIdentifier (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitcoinjs-lib/src/hdnode.js:132:18)
    at HDNode.getFingerprint (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitcoinjs-lib/src/hdnode.js:136:15)
    at HDNode.derive (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitcoinjs-lib/src/hdnode.js:274:31)
    at deriveFast (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitgo/src/bitcoin.js:54:19)
    at derive (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitgo/src/bitcoin.js:159:17)
    at derive (/Applications/Brave.app/Contents/Resources/app.asar/node_modules/bitgo/src/bitcoin.js:147:21)
@bsclifton bsclifton added this to the 0.14.0 milestone Mar 24, 2017
@bsclifton
Copy link
Member Author

Assigning @bridiver as this may be related to the openssl => boringssl switchover

@bridiver
Copy link
Collaborator

I should be able to patch boringssl for this, but can you check the bitcoin library to see if it uses any other crypto methods?

@bridiver
Copy link
Collaborator

fixed brave/muon@1500a27

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants