X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fasync.ts;h=3d6e388096b1a91dbcaa3c1a63aeab57885cae59;hb=3c79c2ce86eaf9e151ab6c2c9d1f646968a16744;hp=f770bc1209e27996e37f0c7fe51a6046df4f1fc1;hpb=90d4bb8125e80c8060416d4d135ddeaf0a622ede;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/middlewares/async.ts b/server/middlewares/async.ts index f770bc120..3d6e38809 100644 --- a/server/middlewares/async.ts +++ b/server/middlewares/async.ts @@ -1,17 +1,19 @@ import { eachSeries } from 'async' import { NextFunction, Request, RequestHandler, Response } from 'express' +import { ValidationChain } from 'express-validator' +import { ExpressPromiseHandler } from '@server/types/express' import { retryTransactionWrapper } from '../helpers/database-utils' // Syntactic sugar to avoid try/catch in express controllers // Thanks: https://medium.com/@Abazhenov/using-async-await-in-express-with-node-8-b8af872c0016 -export type RequestPromiseHandler = (req: Request, res: Response, next: NextFunction) => Promise +export type RequestPromiseHandler = ValidationChain | ExpressPromiseHandler function asyncMiddleware (fun: RequestPromiseHandler | RequestPromiseHandler[]) { return (req: Request, res: Response, next: NextFunction) => { if (Array.isArray(fun) === true) { return eachSeries(fun as RequestHandler[], (f, cb) => { - Promise.resolve(f(req, res, cb)) + Promise.resolve(f(req, res, err => cb(err))) .catch(err => next(err)) }, next) } @@ -21,7 +23,7 @@ function asyncMiddleware (fun: RequestPromiseHandler | RequestPromiseHandler[]) } } -function asyncRetryTransactionMiddleware (fun: RequestPromiseHandler) { +function asyncRetryTransactionMiddleware (fun: (req: Request, res: Response, next: NextFunction) => Promise) { return (req: Request, res: Response, next: NextFunction) => { return Promise.resolve( retryTransactionWrapper(fun, req, res, next)