Handled mongoose Errors as BAD_REQUEST

MiaoMii · Dec 19, 2020 · 50e2a9c · 50e2a9c
1 parent a53bd90
commit 50e2a9c
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/src/middlewares/error.js b/src/middlewares/error.js
@@ -1,3 +1,4 @@
+const mongoose = require('mongoose');
 const httpStatus = require('http-status');
 const config = require('../config/config');
 const logger = require('../config/logger');
@@ -6,9 +7,10 @@ const ApiError = require('../utils/ApiError');
 const errorConverter = (err, req, res, next) => {
   let error = err;
   if (!(error instanceof ApiError)) {
-    const statusCode = error.statusCode || httpStatus.INTERNAL_SERVER_ERROR;
+    const statusCode =
+      error.statusCode || error instanceof mongoose.Error ? httpStatus.BAD_REQUEST : httpStatus.INTERNAL_SERVER_ERROR;
     const message = error.message || httpStatus[statusCode];
-    error = new ApiError(statusCode, message, false, err.stack);
+    error = new ApiError(statusCode, message, !(statusCode === httpStatus.INTERNAL_SERVER_ERROR), err.stack);
   }
   next(error);
 };