Can we support async middleware factories? about meddleware HOT 5 CLOSED

There are two pieces of complexity here.

The first is minor and has to do with a bit of ambiguity. The current signature includes passing in an array of arguments. This signature would have to change to support a callback being programmatically appended. Not a huge deal but, off the top of my head, I cannot come up with a new signature that I like to support both. Love to hear if you have any thoughts.

The second complexity is a bit of a doozy. Express has no concept of asynchronous middleware registration. If you take a peek at how express registers functions into the continuation, it's wholly synchronous. The way things work right now, the moment you register meddleware into your app, everything else done by meddleware happens synchronously. This means that the order of middleware registration is predictable. Introducing async resolution into a pipeline that has no concept of async would mean registration order would no longer be predictable.

In other words, app.use(meddleware(config), otherMiddleware) could result in otherMiddleware being before, after, or somewhere in the middle of meddleware-defined middlewares.

from meddleware.

As for how you can accomplish this already:

The meddleware config can be resolved asynchronously, which would then be passed to meddleware. This is exactly how kraken uses it. To perform a similar action in kraken, you could perform your various async steps in the onconfig method you can pass into the kraken config. You could then set the values resulted by the async steps into the config as middleware arguments.

from meddleware.

Oh wow, haven't thought of that, good point 😄
I guess I can try to use onconfig for now.
Thanks!

from meddleware.

Sure. Let me know if an example or two would help. 😀

from meddleware.

@jasisk I wouldn't mind seeing a couple of examples

from meddleware.