fix(bcrypt tool): Fix bcrypt error states and crashes (#1133) #1152

lionel-rowe · 2024-06-07T12:57:16Z

Started out with fixing #1133, but I also found out it was super easy to crash the browser tab by setting the salt count to a high enough value. In light of that:

Switched from sync to async versions of the bcrypt functions.
Reduced max rounds to 20, as 100 is absurdly high (even 20 is absurdly high with current computing power).
I also added a timeout if hashing/comparing takes >10 seconds. That timeout kicks in somewhere between 16 and 17 rounds on my machine.
Updated visual progress/error states in the UI, including a progress bar and a message showing how long the function took to run.
In-progress functions are aborted if the user changes the inputs before they run to completion.

vercel · 2024-06-07T12:57:20Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
it-tools	✅ Ready (Inspect)	Visit Preview	Aug 9, 2024 0:19am

lionel-rowe · 2024-06-07T13:07:13Z

src/tools/bcrypt/bcrypt.vue

    </n-form>
  </c-card>
 </template>

-<style lang="less" scoped>
+<style lang="less">


Couldn't get the styles to work due to CSS precedence rules while retaining scoped, so I passed an id to c-input-text, which is set on the <input> itself, allowing easier CSS targeting. It's not a great solution though... Open to suggestions of how to do it better without resorting to global styles or !important. Maybe c-input-text just needs finer-grained ways of styling it?