Use response handler in handleErrors middleare

Server/controllers/authController.js

@@ -153,7 +153,9 @@ class AuthController {
 			// Compare password
 			const match = await user.comparePassword(password);
 			if (match !== true) {
-				next(new Error(errorMessages.AUTH_INCORRECT_PASSWORD));
+				const error = new Error(errorMessages.AUTH_INCORRECT_PASSWORD);
+				error.status = 401;
+				next(error);
 				return;
 			}
 
@@ -182,6 +184,7 @@ class AuthController {
 				},
 			});
 		} catch (error) {
+			error.status = 401;
 			next(handleError(error, SERVICE_NAME, "loginUser"));
 		}
 	};

Server/controllers/settingsController.js

@@ -34,7 +34,7 @@ class SettingsController {
 			await this.db.updateAppSettings(req.body);
 			const updatedSettings = { ...(await this.settingsService.reloadSettings()) };
 			delete updatedSettings.jwtSecret;
-			return res.successs({
+			return res.success({
 				msg: successMessages.UPDATE_APP_SETTINGS,
 				data: updatedSettings,
 			});

Server/middleware/handleErrors.js

@@ -11,7 +11,10 @@ const handleErrors = (error, req, res, next) => {
 		method: error.method,
 		stack: error.stack,
 	});
-	res.status(status).json({ success: false, msg: message });
+	res.error({
+		status,
+		msg: message,
+	});
 };
 
 export { handleErrors };

Server/middleware/responseHandler.js

@@ -7,8 +7,8 @@ const responseHandler = (req, res, next) => {
 		});
 	};
 
-	res.error = (status = 500, msg = "Internal server erro", data = null) => {
-		res.status(status).json({
+	res.error = ({ status = 500, msg = "Internal server erro", data = null }) => {
+		return res.status(status).json({
 			success: false,
 			msg,
 			data,