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

feat: handle HTTP 429 as a response code

Browse Source
This commit is contained in:
Julian Lam
2021-01-07 14:05:23 -05:00
parent 56f929ed4f
commit 8bbb320867
2 changed files with 15 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
src/controllers/helpers.js
View File

@@ -461,6 +461,11 @@ helpers.generateError = (statusCode, message) => {
payload.status.message = message || 'HTTPS is required for requests to the write api, please re-send your request via HTTPS';
break;
case 429:
payload.status.code = 'too-many-requests';
payload.status.message = message || 'You have made too many requests, please try again later';
break;
case 500:
payload.status.code = 'internal-server-error';
payload.status.message = message || payload.status.message;

11
src/controllers/write/utilities.js
View File

@@ -27,7 +27,16 @@ Utilities.login = (req, res) => {
const userslug = slugify(username);
const uid = await user.getUidByUserslug(userslug);
const ok = await user.isPasswordCorrect(uid, password, req.ip);
let ok = false;
try {
ok = await user.isPasswordCorrect(uid, password, req.ip);
} catch (err) {
if (err.message === '[[error:account-locked]]') {
helpers.formatApiResponse(429, res, err);
} else {
helpers.formatApiResponse(500, res, err);
}
}
if (ok) {
const userData = await user.getUsers([uid], uid);