X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fvalidators%2Fsearch.ts;h=e6ec8642ada15f3e262c4e49fc12dfba4dc6ce78;hb=7e0f50d6e0c7dc583d40e196c283eb20dc386ae6;hp=b4faa8894ab8620330f9187514f4bb5ef3244d1b;hpb=3b0bd70aa05ab82fa30fe67ed4899d44652c703a;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/middlewares/validators/search.ts b/server/middlewares/validators/search.ts index b4faa8894..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('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 valid published start date'), - query('originallyPublishedEndDate').optional().custom(isDateValid).withMessage('Should have a valid published end date'), + 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('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'), @@ -28,9 +51,23 @@ const videosSearchValidator = [ } ] -const videoChannelsSearchValidator = [ - query('search').not().isEmpty().withMessage('Should have a valid search'), - query('searchTarget').optional().custom(isSearchTargetValid).withMessage('Should have a valid search target'), +const videoChannelsListSearchValidator = [ + 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,9 +78,38 @@ const videoChannelsSearchValidator = [ } ] +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 playlists search query', { parameters: req.query }) + + if (areValidationErrors(req, res)) return + + return next() + } +] + // --------------------------------------------------------------------------- export { - videoChannelsSearchValidator, - videosSearchValidator + videosSearchValidator, + videoChannelsListSearchValidator, + videoPlaylistsListSearchValidator }