X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fasync.ts;h=f770bc1209e27996e37f0c7fe51a6046df4f1fc1;hb=6ca76832fde247a33cb2ee94208f492801d1791a;hp=29ebd169d4a3e911adc2f86a22226394fcc4068b;hpb=eb08047657e739bcd9e592d76307befa3998482b;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/middlewares/async.ts b/server/middlewares/async.ts index 29ebd169d..f770bc120 100644 --- a/server/middlewares/async.ts +++ b/server/middlewares/async.ts @@ -1,16 +1,37 @@ -import { Request, Response, NextFunction } from 'express' +import { eachSeries } from 'async' +import { NextFunction, Request, RequestHandler, Response } from '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 -function asyncMiddleware (fn: (req: Request, res: Response, next: NextFunction) => Promise) { + +export type RequestPromiseHandler = (req: Request, res: Response, next: NextFunction) => Promise + +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)) + .catch(err => next(err)) + }, next) + } + + return Promise.resolve((fun as RequestHandler)(req, res, next)) + .catch(err => next(err)) + } +} + +function asyncRetryTransactionMiddleware (fun: RequestPromiseHandler) { return (req: Request, res: Response, next: NextFunction) => { - return Promise.resolve(fn(req, res, next)) - .catch(next) + return Promise.resolve( + retryTransactionWrapper(fun, req, res, next) + ).catch(err => next(err)) } } // --------------------------------------------------------------------------- export { - asyncMiddleware + asyncMiddleware, + asyncRetryTransactionMiddleware }