search

keeweb / kdbxweb

Web Kdbx library
https://app.keeweb.info
MIT License
409 stars 57 forks source link

Performance #17

Closed thardes2 closed 6 years ago

thardes2 commented 6 years ago

I was using the version 0.4.6 until you fixed issue #16 yesterday (thank you :-)).

I noticed some changes in the performance, as the latest repository version is about three times slower than version 0.4.6: image

Well, three times slower is quite high :-), do you plan to improve this or is there any way to make it faster?

antelle commented 6 years ago

Could you please attach an example of file to check where it hangs? Is it a complex hash function with many rounds, or the file is large?

thardes2 commented 6 years ago

Well, unfortunately I cannot give the file I used for the screenshots above, but I created a new one, which is uploaded in this issue. Temp12345.zip The zip file is not encrypted, and the password for the kdbx-file is "Temp12345"

However, the effect is similar: 310 ms in version 0.4.6 and 551 ms in the latest version

antelle commented 6 years ago

Good catch! It turns out, TextEncoding polyfill was replacing native TextEncoder even if was supported in browser, now it's much faster, will release it later today. 🎁

antelle commented 6 years ago

Published v1.0.2