Merge pull request #1597 from TriliumNext/hotfix/totp-validation-bypass

hotfix(auth): fix TOTP validation bypass issue
This commit is contained in:
JYC333 2025-04-02 11:42:38 +02:00 committed by GitHub
commit 9f3076755c
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -77,11 +77,6 @@ function login(req: Request, res: Response) {
const submittedPassword = req.body.password;
const submittedTotpToken = req.body.totpToken;
if (!verifyPassword(submittedPassword)) {
sendLoginError(req, res, 'password');
return;
}
if (totp.isTotpEnabled()) {
if (!verifyTOTP(submittedTotpToken)) {
sendLoginError(req, res, 'totp');
@ -89,6 +84,11 @@ function login(req: Request, res: Response) {
}
}
if (!verifyPassword(submittedPassword)) {
sendLoginError(req, res, 'password');
return;
}
const rememberMe = req.body.rememberMe;
req.session.regenerate(() => {