X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fvalidators%2Fsearch.ts;h=e6ec8642ada15f3e262c4e49fc12dfba4dc6ce78;hb=ced38c0ffe1e7d30f1f80fe704e571f39b0cc78b;hp=78213c70da16a02a163f63eb85489c541c5915bc;hpb=7b3909644dd7cb8be1caad537bb40605e5f059d4;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/middlewares/validators/search.ts b/server/middlewares/validators/search.ts index 78213c70d..e6ec8642a 100644 --- a/server/middlewares/validators/search.ts +++ b/server/middlewares/validators/search.ts @@ -1,21 +1,44 @@ -import * as express from 'express' -import { areValidationErrors } from './utils' -import { logger } from '../../helpers/logger' +import express from 'express' import { query } from 'express-validator' -import { isDateValid } from '../../helpers/custom-validators/misc' import { isSearchTargetValid } from '@server/helpers/custom-validators/search' +import { isHostValid } from '@server/helpers/custom-validators/servers' +import { areUUIDsValid, isDateValid, isNotEmptyStringArray, toCompleteUUIDs } from '../../helpers/custom-validators/misc' +import { logger } from '../../helpers/logger' +import { areValidationErrors } from './shared' const videosSearchValidator = [ query('search').optional().not().isEmpty().withMessage('Should have a valid search'), - query('startDate').optional().custom(isDateValid).withMessage('Should have a valid start date'), - query('endDate').optional().custom(isDateValid).withMessage('Should have a valid end date'), - - query('originallyPublishedStartDate').optional().custom(isDateValid).withMessage('Should have a valid published start date'), - query('originallyPublishedEndDate').optional().custom(isDateValid).withMessage('Should have a valid published end date'), - - query('durationMin').optional().isInt().withMessage('Should have a valid min duration'), - query('durationMax').optional().isInt().withMessage('Should have a valid max duration'), + query('host') + .optional() + .custom(isHostValid).withMessage('Should have a valid host'), + + query('startDate') + .optional() + .custom(isDateValid).withMessage('Should have a start date that conforms to ISO 8601'), + query('endDate') + .optional() + .custom(isDateValid).withMessage('Should have a end date that conforms to ISO 8601'), + + query('originallyPublishedStartDate') + .optional() + .custom(isDateValid).withMessage('Should have a published start date that conforms to ISO 8601'), + query('originallyPublishedEndDate') + .optional() + .custom(isDateValid).withMessage('Should have a published end date that conforms to ISO 8601'), + + query('durationMin') + .optional() + .isInt().withMessage('Should have a valid min duration'), + query('durationMax') + .optional() + .isInt().withMessage('Should have a valid max duration'), + + query('uuids') + .optional() + .toArray() + .customSanitizer(toCompleteUUIDs) + .custom(areUUIDsValid).withMessage('Should have valid uuids'), query('searchTarget').optional().custom(isSearchTargetValid).withMessage('Should have a valid search target'), @@ -29,8 +52,22 @@ const videosSearchValidator = [ ] const videoChannelsListSearchValidator = [ - query('search').not().isEmpty().withMessage('Should have a valid search'), - query('searchTarget').optional().custom(isSearchTargetValid).withMessage('Should have a valid search target'), + query('search') + .optional() + .not().isEmpty().withMessage('Should have a valid search'), + + query('host') + .optional() + .custom(isHostValid).withMessage('Should have a valid host'), + + query('searchTarget') + .optional() + .custom(isSearchTargetValid).withMessage('Should have a valid search target'), + + query('handles') + .optional() + .toArray() + .custom(isNotEmptyStringArray).withMessage('Should have valid handles'), (req: express.Request, res: express.Response, next: express.NextFunction) => { logger.debug('Checking video channels search query', { parameters: req.query }) @@ -41,11 +78,27 @@ const videoChannelsListSearchValidator = [ } ] -const videoChannelsOwnSearchValidator = [ - query('search').optional().not().isEmpty().withMessage('Should have a valid search'), +const videoPlaylistsListSearchValidator = [ + query('search') + .optional() + .not().isEmpty().withMessage('Should have a valid search'), + + query('host') + .optional() + .custom(isHostValid).withMessage('Should have a valid host'), + + query('searchTarget') + .optional() + .custom(isSearchTargetValid).withMessage('Should have a valid search target'), + + query('uuids') + .optional() + .toArray() + .customSanitizer(toCompleteUUIDs) + .custom(areUUIDsValid).withMessage('Should have valid uuids'), (req: express.Request, res: express.Response, next: express.NextFunction) => { - logger.debug('Checking video channels search query', { parameters: req.query }) + logger.debug('Checking video playlists search query', { parameters: req.query }) if (areValidationErrors(req, res)) return @@ -58,5 +111,5 @@ const videoChannelsOwnSearchValidator = [ export { videosSearchValidator, videoChannelsListSearchValidator, - videoChannelsOwnSearchValidator + videoPlaylistsListSearchValidator }