X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fmiddlewares%2Fvalidators%2Fvideos%2Fvideo-watch.ts;h=d6ca1d3412064371c13bd41ca0b09f64355055f0;hb=610d0be13b3d01f653ef269271dd667a57c85ef2;hp=bca64662fbf9a0814a0af1dabd462915efcaf43a;hpb=be0f59b4eec3c2c4dcd151e2b174be39dff1568e;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/middlewares/validators/videos/video-watch.ts b/server/middlewares/validators/videos/video-watch.ts index bca64662f..d6ca1d341 100644 --- a/server/middlewares/validators/videos/video-watch.ts +++ b/server/middlewares/validators/videos/video-watch.ts @@ -1,21 +1,27 @@ -import { body, param } from 'express-validator/check' +import { body, param } from 'express-validator' import * as express from 'express' -import { isIdOrUUIDValid } from '../../../helpers/custom-validators/misc' -import { isVideoExist } from '../../../helpers/custom-validators/videos' +import { isIdOrUUIDValid, toIntOrNull } from '../../../helpers/custom-validators/misc' import { areValidationErrors } from '../utils' import { logger } from '../../../helpers/logger' +import { doesVideoExist } from '../../../helpers/middlewares' const videoWatchingValidator = [ param('videoId').custom(isIdOrUUIDValid).not().isEmpty().withMessage('Should have a valid id'), body('currentTime') - .toInt() + .customSanitizer(toIntOrNull) .isInt().withMessage('Should have correct current time'), async (req: express.Request, res: express.Response, next: express.NextFunction) => { logger.debug('Checking videoWatching parameters', { parameters: req.body }) if (areValidationErrors(req, res)) return - if (!await isVideoExist(req.params.videoId, res, 'id')) return + if (!await doesVideoExist(req.params.videoId, res, 'id')) return + + const user = res.locals.oauth.token.User + if (user.videosHistoryEnabled === false) { + logger.warn('Cannot set videos to watch by user %d: videos history is disabled.', user.id) + return res.status(409).end() + } return next() }