Fix jsbn browser check#1106
Conversation
Previous versions of NodeJS/Bun/Deno didn't have 'navigator' defined, while latest one do have it, which forces am2 algorithm with 26 dbits, which is awfully slow, at least in NodeJS v22 (compared to am3 with 28 bits)
|
@davidlehn this PR is an alternative to the PR by @dhensby |
|
This feels like a bad idea, appName is deprecated, so liable to be removed from browsers - meaning that it could cause unintended behaviour in browsers. Also, we're still coupling with the |
|
Thanks for the comment, I will close my PR then. |
fixes #1081
Previous versions of NodeJS/Bun/Deno didn't have 'navigator' defined, while latest one do have it, which forces am2 algorithm with 26 dbits, which is awfully slow (~20x), at least in NodeJS v22 (compared to am3 with 28 dbits)