Skip to content

Conversation

@Linkan333
Copy link

Description

I increased the ITERATIONS variable in the decrypt_email.js file. This is because it could be usefull for Brute-Force Attacks to use PBKDF2 (Password-Bases Key Derivation Function 2). It could be bad for the performance on your website cause it will take longer time for each key to be generated.

For example at 100,000 iterations it might take ~100ms to derive a key. If you do the math with approximate 1 million password it would take ~27 hours on a single cpu to crack

…will lower the performace this is not a must to add but could be good
@vercel
Copy link

vercel bot commented Apr 16, 2025

@Linkan333 is attempting to deploy a commit to the Revert Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions
Copy link

Hey there and thank you for opening this pull request! 👋🏼
We require pull request titles to follow the Conventional Commits specification and it looks like your proposed title needs to be adjusted.
Details:

No release type found in pull request title "Increased ITERATIONS in decrypt_email.js". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/

Available types:
 - feat: A new feature
 - fix: A bug fix
 - docs: Documentation only changes
 - style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
 - refactor: A code change that neither fixes a bug nor adds a feature
 - perf: A code change that improves performance
 - test: Adding missing tests or correcting existing tests
 - build: Changes that affect the build system or external dependencies (example scopes: gulp, broccoli, npm)
 - ci: Changes to our CI configuration files and scripts (example scopes: Travis, Circle, BrowserStack, SauceLabs)
 - chore: Other changes that don't modify src or test files
 - revert: Reverts a previous commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant