-
Notifications
You must be signed in to change notification settings - Fork 22
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
Password generated in Edge is different to other implementations #8
Comments
Can you post the output of https://tmthrgd.github.io/mpw-js/test/? |
@leemachin That error shouldn't have prevented the test suite from functioning, but either way, it has now been fixed (see #9). Would you mind running the test suite again for me? |
was curious to help and check in my dual core W10-64 pro 1809 system. MS Edge 44.17763.1.0 / EdgeHTML 18.17763 completely fails right away. (seems like Fx doesn't soak up the cpu power until almost the end). |
@jmichael2497 Thanks for the update. Is there any error message from the failure or in the debug console? |
@jmichael2497 (From memory): The slowdown in Firefox is caused by missing support for the right PBKDF2 algorithm which is instead done partly in JavaScript. |
sorry forgot to specify all test lines just end basically with similar variable timings of "Failed; [object Object] (after 601.4000000000001ms)" and a quick peek through the built in dev tools did not show any errors or warnings in the console. i would generally recommend folks use a clean and locked down browser like fx or gc anyway (shrug). |
for reference on same dual core W10-64 pro 21H2 system:
|
In Edge 14:
In Safari/Chrome + Non-web versions:
The text was updated successfully, but these errors were encountered: