Update index.d.ts

[juliusmarminge]

excuse me if this is invalid but from reading the examples it seems like async functions are allowed, so the return type should accept a promise?

i've been thrown off from all the synchronous types in Fastify the last couple of days, so perhaps this should be looked over in other parts of the ecosystem too

[mcollina]

What examples? Are those on our website?

You really should not use async here, nor in most other middleware, because there is no real error handling for rejecting promises. Nor there is in express or connect etc.

In essence, the types are telling you that what you are doing is essentially unsafe, because it is.

[Uzlopak]

In express v4 I needed to use this one "hack" where you could have async handlers. I think express v5 has native async handler support.

[mcollina]

Unfortunately we do not handle promises here. A rejection in one of those promise would lead to a unhandledRejection, therefore making it problematic. We could support this, but work to add it should be done to library, not just the types.