Skip to content
This repository has been archived by the owner on Jan 25, 2020. It is now read-only.

Feature: Support for Async Middleware #98

Open
wants to merge 1 commit into
base: v1.x
Choose a base branch
from

Conversation

xjamundx
Copy link

@xjamundx xjamundx commented Mar 19, 2018

Let's officially add support for async middleware / routes.

Does this by wrapping all routes at the enrouten level with a promise and catching errors and forwarding them to next(err)

Have been using this internally, but thought it might be cool to make it public. I'd suggest major bumping this and releasing it as a 2.0 enhancement, but outside of the use of the spread operator it should work in node 4+.

@aheckmann @snowinferno @shaunwarman

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant