diff options
author | Chocobozzz <me@florianbigard.com> | 2020-01-09 09:36:31 +0100 |
---|---|---|
committer | Chocobozzz <me@florianbigard.com> | 2020-01-09 09:36:31 +0100 |
commit | e0b56b7495e809581a1e6447794bf7573a78af56 (patch) | |
tree | 7bf3917b7109fc8d671c994b4f501fd020257b38 /server/middlewares/validators/activitypub/pagination.ts | |
parent | 240458d0c95133bfb03a3f695a1b4e5cb63e20ef (diff) | |
download | PeerTube-e0b56b7495e809581a1e6447794bf7573a78af56.tar.gz PeerTube-e0b56b7495e809581a1e6447794bf7573a78af56.tar.zst PeerTube-e0b56b7495e809581a1e6447794bf7573a78af56.zip |
Return an error on invalid count pagination
Diffstat (limited to 'server/middlewares/validators/activitypub/pagination.ts')
-rw-r--r-- | server/middlewares/validators/activitypub/pagination.ts | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/server/middlewares/validators/activitypub/pagination.ts b/server/middlewares/validators/activitypub/pagination.ts index 8b32d3415..fa21f063d 100644 --- a/server/middlewares/validators/activitypub/pagination.ts +++ b/server/middlewares/validators/activitypub/pagination.ts | |||
@@ -2,10 +2,15 @@ import * as express from 'express' | |||
2 | import { query } from 'express-validator' | 2 | import { query } from 'express-validator' |
3 | import { logger } from '../../../helpers/logger' | 3 | import { logger } from '../../../helpers/logger' |
4 | import { areValidationErrors } from '../utils' | 4 | import { areValidationErrors } from '../utils' |
5 | import { PAGINATION } from '@server/initializers/constants' | ||
5 | 6 | ||
6 | const apPaginationValidator = [ | 7 | const apPaginationValidator = [ |
7 | query('page').optional().isInt({ min: 1 }).withMessage('Should have a valid page number'), | 8 | query('page') |
8 | query('size').optional().isInt({ max: 50 }).withMessage('Should have a valid page size (max: 50)'), | 9 | .optional() |
10 | .isInt({ min: 1 }).withMessage('Should have a valid page number'), | ||
11 | query('size') | ||
12 | .optional() | ||
13 | .isInt({ min: 0, max: PAGINATION.OUTBOX.COUNT.MAX }).withMessage(`Should have a valid page size (max: ${PAGINATION.OUTBOX.COUNT.MAX})`), | ||
9 | 14 | ||
10 | (req: express.Request, res: express.Response, next: express.NextFunction) => { | 15 | (req: express.Request, res: express.Response, next: express.NextFunction) => { |
11 | logger.debug('Checking pagination parameters', { parameters: req.query }) | 16 | logger.debug('Checking pagination parameters', { parameters: req.query }) |