X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fvalidators%2Futils.ts;h=8f77c9fbd84060118df26ab1a6b8607b19d28f7a;hb=f0a3988066f72a28bb44520af072f18d91d77dde;hp=0424d59429f6e914fa7f855db71afcb1af4519cb;hpb=69818c9394366b954b6ba3bd697bd9d2b09f2a16;p=github%2FChocobozzz%2FPeerTube.git

diff --git a/server/middlewares/validators/utils.ts b/server/middlewares/validators/utils.ts
index 0424d5942..8f77c9fbd 100644
--- a/server/middlewares/validators/utils.ts
+++ b/server/middlewares/validators/utils.ts
@@ -1,22 +1,44 @@
-import 'express-validator'
 import * as express from 'express'
-import { inspect } from 'util'
+import { query, validationResult } from 'express-validator/check'
+import { logger } from '../../helpers/logger'
 
-import { logger } from '../../helpers'
+function areValidationErrors (req: express.Request, res: express.Response) {
+  const errors = validationResult(req)
 
-function checkErrors (req: express.Request, res: express.Response, next: express.NextFunction, statusCode = 400) {
-  const errors = req.validationErrors()
+  if (!errors.isEmpty()) {
+    logger.warn('Incorrect request parameters', { path: req.originalUrl, err: errors.mapped() })
+    res.status(400).json({ errors: errors.mapped() })
 
-  if (errors) {
-    logger.warn('Incorrect request parameters', { path: req.originalUrl, err: errors })
-    return res.status(statusCode).send('There have been validation errors: ' + inspect(errors))
+    return true
   }
 
-  return next()
+  return false
+}
+
+function checkSort (sortableColumns: string[]) {
+  return [
+    query('sort').optional().isIn(sortableColumns).withMessage('Should have correct sortable column'),
+
+    (req: express.Request, res: express.Response, next: express.NextFunction) => {
+      logger.debug('Checking sort parameters', { parameters: req.query })
+
+      if (areValidationErrors(req, res)) return
+
+      return next()
+    }
+  ]
+}
+
+function createSortableColumns (sortableColumns: string[]) {
+  const sortableColumnDesc = sortableColumns.map(sortableColumn => '-' + sortableColumn)
+
+  return sortableColumns.concat(sortableColumnDesc)
 }
 
 // ---------------------------------------------------------------------------
 
 export {
-  checkErrors
+  areValidationErrors,
+  checkSort,
+  createSortableColumns
 }