-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Doesn't work in Node.js #2
Comments
Replace the following (from line 100 of node_modules/pidcrypt/jsbn.js):
With:
|
@franciskim 's suggestion worked, but come on seriously, can't this dependency be swapped for another one? |
Suggestions more than welcome @kevinkga. Please feel free to find an alternative encryption library and submit a PR. |
I've requested @franciskim fix to be merged into pidcrypt nikvdp/pidcrypt#7. @franciskim please let me know if you want some mention or credit for that. I just want to get it fixed. |
Due to the use of pidcrypt, I currently get the error below when running in a Node.js environment:
if(j_lm && (navigator.appName == "Microsoft Internet Explorer")) {
________________^
ReferenceError: navigator is not defined
Also, it doesnt look like something that will be fixed: nikvdp/pidcrypt#5
The text was updated successfully, but these errors were encountered: