Support Async Middleware about kraken-js

[dougwilson]

Perhaps Express can add a hook to register a function to wrap middleware, which could be used to automatically wrap the middleware / handers for this and whatever other purposes. The details for how the interaction will be done with routers will need to be hammered out, but a high-level idea is:

app.wrap(function (fn) {
  return function (req, res, next) {
    fn(req, res, next).catch(next)
  }
})

This is very high-level thought, but figured I'd start a discussion here from the comment in Express repo to get a feeling of if this could be a solution or not.

[xjamundx]

@dougwilson I think at this stage with async/await finally built-in to the platform, it should just become a core express/kraken feature to support promise-based middleware (without even having to opt-in via app.wrap).

It sounds like express 5 is stalling pretty hard (which is fine), so our best bet will be to add this support directly into kraken I think.

[dougwilson]

Yea, and it should, but there are still unresolved issues on how exactly we should deal with the resolved values, for example, and how to deal with the Bluebird warnings, as another.

[xjamundx]

On the kraken side there a few ways to implement this. One place we may want to look at is in express-enrouten such as here:
https://github.com/krakenjs/express-enrouten/blob/v1.x/lib/directory.js#L69

[xjamundx]

Another thing we could do is put this monkey-patching script into its own module and just require it in as needed (either in kraken core or as wanted by various teams):
https://gist.github.com/q42jaap/f2fb93d96fda6384d3e3fc51977dec90

[xjamundx]

We could potentially integrate with this module:
https://github.com/davidbanham/express-async-errors