Error Handling Middleware about bluewave-uptime

[ajhollid]

@veyselboybay what do you think about this one? Say we have a middleware like

app.use((err, req, res, next) => {
  const statusCode = err.statusCode || 500;
  logger.error(err.message, { service: error.service });
  res.status(statusCode).json({ success: false, msg: err.message });
});

We can pass any errors from the controllers/other middleware to the error handler:

try{
  ...
} catch (error) {
    error.statusCode = 404
    error.service = "Auth Service"
    next(error)
}

Something like that?

We can cut down on some repeated code in the controllers/middleware and keep error handling centralized as well

[ajhollid]

Sure, I'll try putting it together and see how it works

[veyselboybay]

This looks like a great idea and seems like the best practice. Have you got a chance to test it?