Nemcio/NodeBB
1
0
Fork 0
mirror of https://github.com/NodeBB/NodeBB.git synced 2025-10-26 16:46:12 +01:00
Code Issues Packages Projects Releases Wiki Activity

fix: login handler to handle if non-confirmed email is entered

Browse Source
This commit is contained in:
Julian Lam
2025-10-01 13:51:04 -04:00
parent 56a9336611
commit 5ed19ef8a9
2 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/controllers/authentication.js
View File

@@ -262,6 +262,8 @@ authenticationController.login = async (req, res, next) => {
const username = await user.getUsernameByEmail(req.body.username);
if (username !== '[[global:guest]]') {
req.body.username = username;
} else {
return errorHandler(req, res, '[[error:invalid-email]]', 400);
}
}
if (isEmailLogin || isUsernameLogin) {

6
test/authentication.js
View File

@@ -5,8 +5,8 @@ const assert = require('assert');
const url = require('url');
const nconf = require('nconf');
const request = require('../src/request');
const db = require('./mocks/databasemock');
const request = require('../src/request');
const user = require('../src/user');
const utils = require('../src/utils');
const meta = require('../src/meta');
@@ -52,8 +52,8 @@ describe('authentication', () => {
meta.config.allowLoginWith = 'username-email';
const uid = await user.create({ username: '2nduser', password: '2ndpassword', email: '2nduser@nodebb.org' });
const { response, body } = await helpers.loginUser('2nduser@nodebb.org', '2ndpassword');
assert.strictEqual(response.statusCode, 403);
assert.strictEqual(body, '[[error:invalid-login-credentials]]');
assert.strictEqual(response.statusCode, 400);
assert.strictEqual(body, '[[error:invalid-email]]');
meta.config.allowLoginWith = oldValue;
});