add global api error handler

v1/src/app.js

@@ -11,6 +11,7 @@ const {
 } = require("./routes");
 const events = require("./scripts/events");
 const path = require("path");
+const errorHandler = require("./middlewares/errorHandler");
 
 config();
 loaders();
@@ -28,4 +29,12 @@ app.listen(process.env.APP_PORT, () => {
   app.use("/users", UserRoutes);
   app.use("/sections", SectionRoutes);
   app.use("/tasks", TaskRoutes);
+
+  app.use((req, res, next) => {
+    const error = new Error("Not found");
+    error.status = 404;
+    next(error);
+  });
+
+  app.use(errorHandler);
 });

v1/src/controllers/Projects.js

@@ -1,5 +1,6 @@
 const httpStatus = require("http-status");
 const projectService = require("../services/ProjectService");
+const ApiError = require("../errors/ApiError");
 
 class ProjectsController {
   index(req, res) {
@@ -25,7 +26,7 @@ class ProjectsController {
       });
   }
 
-  update(req, res) {
+  update(req, res, next) {
     if (!req.params?.id)
       return res
         .status(httpStatus.NOT_FOUND)
@@ -34,6 +35,9 @@ class ProjectsController {
     projectService
       .update(req.body, req.params.id)
       .then((updatedProject) => {
+        if (!updatedProject)
+          /*throw new ApiError("Project not found", 404);*/
+          return next(new ApiError("Project not found", 404));
         res.status(httpStatus.OK).send(updatedProject);
       })
       .catch((e) => {

v1/src/errors/ApiError.js

@@ -0,0 +1,9 @@
+class ApiError extends Error {
+  constructor(message, status) {
+    super(message);
+    this.status = status;
+    this.message = message;
+  }
+}
+
+module.exports = ApiError;

v1/src/middlewares/errorHandler.js

@@ -0,0 +1,8 @@
+module.exports = (error, req, res, next) => {
+  res.status(error.status || 500);
+  res.json({
+    error: {
+      message: error.message || "Internal Server Error...",
+    },
+  });
+};