navigator is not defined

[kYuZz]

To reproduce, create an empty file and save it with the name index.js:

require('pidcrypt/seedrandom')
const pidCrypt = require('pidcrypt')
require('pidcrypt/rsa')

Running node index.js outputs this error:

node_modules/pidcrypt/jsbn.js:100
if(j_lm && (navigator.appName == "Microsoft Internet Explorer")) {
            ^

ReferenceError: navigator is not defined
    at Object.<anonymous> (/home/kyuzz/Projects/test-ig-api/node_modules/pidcrypt/jsbn.js:100:13)
    at Module._compile (module.js:556:32)
    at Object.Module._extensions..js (module.js:565:10)
    at Module.load (module.js:473:32)
    at tryModuleLoad (module.js:432:12)
    at Function.Module._load (module.js:424:3)
    at Module.require (module.js:483:17)
    at require (internal/module.js:20:19)
    at Object.<anonymous> (/home/kyuzz/Projects/test-ig-api/node_modules/pidcrypt/rsa.js:63:18)
    at Module._compile (module.js:556:32)

[rabbitlair]

Same here, I have the exact same error when using last pidCrypt version.

[kambajwa]

same issue here, any updates ??

[kambajwa]

Any updates on this issue ??

[joseSantacruz]

any update???

[kambajwa]

@joseSantacruz sorry, i'haven't received any updates either but i comment out the

&& (navigator.appName == "Microsoft Internet Explorer")
or you can always use
navigator = {appName:'VM'};

as a temporary fix until you have any thing..

[kevinkga]

It'd be quite easy to fix. I wonder why nobody has bothered? Just swap the following lines

pidcrypt/jsbn.js

Line 100 in cc2b10a

if(j_lm && (navigator.appName == "Microsoft Internet Explorer")) {

with
``
if(j_lm && typeof navigator !== 'undefined' && (navigator.appName == "Microsoft Internet Explorer")) {
BigInteger.prototype.am = am2;
dbits = 30;
}
else if(j_lm && typeof navigator !== 'undefined' && (navigator.appName != "Netscape")) {
BigInteger.prototype.am = am1;
dbits = 26;
}

[ChristophDietrich]

Please make all forkers a happy day, do an update here ;)

Would be awesome! @nikvdp

[nikvdp]

merged!

[esellin]

A new version published on NPM would be awesome!

[kea0811]

@nikvdp hello, will you consider to published the merge version in npm?

[nikvdp]

@kea0811 @esellin published!