Skip to content

Commit

Permalink
Merge pull request #2048 from scireum/feature/fha/SE-14149-Security
Browse files Browse the repository at this point in the history
SE-14149: Fixes client side password checks.
  • Loading branch information
jmuscireum authored Nov 25, 2024
2 parents 752ce97 + bec35fa commit 359caef
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -84,11 +84,11 @@
return false;
}

if (settings.requireLettersAndDigits && !/\d/.test(password) && !/[a-zA-Z]/.test(password)) {
if (settings.requireLettersAndDigits && (!/\d/.test(password) || !/[a-zA-Z]/.test(password))) {
return false;
}

if (settings.requireUpperAndLowerCase && !/[a-z]/.test(password) && !/[A-Z]/.test(password)) {
if (settings.requireUpperAndLowerCase && (!/[a-z]/.test(password) || !/[A-Z]/.test(password))) {
return false;
}

Expand Down

0 comments on commit 359caef

Please sign in to comment.