fix web worker issues

bitcore should be able to run inside a web worker. There were two minor issues
preventing bitcore from running inside a web worker. The first was that lodash
was outdated, and that version of lodash had a problem with web workers. The
second was that the wrong version of ripemd160 was being called, because
global.window does not exist inside a web worker (global.self does instead). A
better way to check if you are in a browser is with process.browser.
This commit is contained in:
Ryan X. Charles 2015-08-20 17:49:20 -07:00
parent aba78f92ba
commit c1087c2827
3 changed files with 5 additions and 5 deletions

View File

@ -34,7 +34,7 @@ Hash.ripemd160 = function(buf) {
// Node.js crypto ripemd160 hashes are not supported in a browser // Node.js crypto ripemd160 hashes are not supported in a browser
// We'll replace with a (slower) version that does. // We'll replace with a (slower) version that does.
if (global.window) { if (process.browser) {
Hash.ripemd160 = function(buf) { Hash.ripemd160 = function(buf) {
$.checkArgument(BufferUtil.isBuffer(buf)); $.checkArgument(BufferUtil.isBuffer(buf));
var hash = (new hashjs.ripemd160()).update(buf).digest(); var hash = (new hashjs.ripemd160()).update(buf).digest();

6
npm-shrinkwrap.json generated
View File

@ -35,9 +35,9 @@
"resolved": "https://registry.npmjs.org/inherits/-/inherits-2.0.1.tgz" "resolved": "https://registry.npmjs.org/inherits/-/inherits-2.0.1.tgz"
}, },
"lodash": { "lodash": {
"version": "2.4.1", "version": "3.10.1",
"from": "lodash@=2.4.1", "from": "lodash@=3.10.1",
"resolved": "https://registry.npmjs.org/lodash/-/lodash-2.4.1.tgz" "resolved": "https://registry.npmjs.org/lodash/-/lodash-3.10.1.tgz"
}, },
"sha512": { "sha512": {
"version": "0.0.1", "version": "0.0.1",

View File

@ -84,7 +84,7 @@
"elliptic": "=3.0.3", "elliptic": "=3.0.3",
"hash.js": "=1.0.2", "hash.js": "=1.0.2",
"inherits": "=2.0.1", "inherits": "=2.0.1",
"lodash": "=2.4.1", "lodash": "=3.10.1",
"sha512": "=0.0.1" "sha512": "=0.0.1"
}, },
"devDependencies": { "devDependencies": {