carbon-io / carbond

MIT License
2 stars 5 forks source link

Error handling middleware is initialized after doStart #257

Closed tfogo closed 6 years ago

tfogo commented 6 years ago

See Service.js#L800. It seems weird to me. I would have expected the order be:

Is there a reason for error handling middleware to be defined after?

gregbanks commented 6 years ago

from express documentation: You define error-handling middleware last, after other app.use() and routes calls