-import * as express from 'express'
+import express from 'express'
import { param, query } from 'express-validator'
-import { isIdOrUUIDValid, isIdValid } from '../../helpers/custom-validators/misc'
-import { logger } from '../../helpers/logger'
-import { areValidationErrors } from './utils'
+import { HttpStatusCode } from '../../../shared/models/http/http-error-codes'
import { isValidRSSFeed } from '../../helpers/custom-validators/feeds'
-import { doesVideoExist } from '../../helpers/middlewares/videos'
+import { exists, isIdOrUUIDValid, isIdValid, toCompleteUUID } from '../../helpers/custom-validators/misc'
+import { logger } from '../../helpers/logger'
import {
+ areValidationErrors,
doesAccountIdExist,
doesAccountNameWithHostExist,
+ doesUserFeedTokenCorrespond,
doesVideoChannelIdExist,
doesVideoChannelNameWithHostExist,
- doesUserFeedTokenCorrespond
-} from '../../helpers/middlewares'
+ doesVideoExist
+} from './shared'
const feedsFormatValidator = [
param('format').optional().custom(isValidRSSFeed).withMessage('Should have a valid format (rss, atom, json)'),
if (req.accepts(acceptableContentTypes)) {
res.set('Content-Type', req.accepts(acceptableContentTypes) as string)
} else {
- return res.status(406).send({
+ return res.fail({
+ status: HttpStatusCode.NOT_ACCEPTABLE_406,
message: `You should accept at least one of the following content-types: ${acceptableContentTypes.join(', ')}`
- }).end()
+ })
}
return next()
}
const videoFeedsValidator = [
- query('accountId').optional().custom(isIdValid),
- query('accountName').optional(),
- query('videoChannelId').optional().custom(isIdValid),
- query('videoChannelName').optional(),
+ query('accountId')
+ .optional()
+ .custom(isIdValid)
+ .withMessage('Should have a valid account id'),
+
+ query('accountName')
+ .optional(),
+
+ query('videoChannelId')
+ .optional()
+ .custom(isIdValid)
+ .withMessage('Should have a valid channel id'),
+
+ query('videoChannelName')
+ .optional(),
async (req: express.Request, res: express.Response, next: express.NextFunction) => {
logger.debug('Checking feeds parameters', { parameters: req.query })
}
]
-const videoSubscriptonFeedsValidator = [
- query('accountId').custom(isIdValid),
- query('token'),
+const videoSubscriptionFeedsValidator = [
+ query('accountId')
+ .custom(isIdValid)
+ .withMessage('Should have a valid account id'),
+
+ query('token')
+ .custom(exists)
+ .withMessage('Should have a token'),
async (req: express.Request, res: express.Response, next: express.NextFunction) => {
- logger.debug('Checking feeds parameters', { parameters: req.query })
+ logger.debug('Checking subscription feeds parameters', { parameters: req.query })
if (areValidationErrors(req, res)) return
- // a token alone is erroneous
- if (req.query.token && !req.query.accountId) return
- if (req.query.accountId && !await doesAccountIdExist(req.query.accountId, res)) return
- if (req.query.token && !await doesUserFeedTokenCorrespond(res.locals.account.userId, req.query.token, res)) return
+ if (!await doesAccountIdExist(req.query.accountId, res)) return
+ if (!await doesUserFeedTokenCorrespond(res.locals.account.userId, req.query.token, res)) return
return next()
}
]
const videoCommentsFeedsValidator = [
- query('videoId').optional().custom(isIdOrUUIDValid),
+ query('videoId')
+ .customSanitizer(toCompleteUUID)
+ .optional()
+ .custom(isIdOrUUIDValid),
async (req: express.Request, res: express.Response, next: express.NextFunction) => {
logger.debug('Checking feeds parameters', { parameters: req.query })
if (areValidationErrors(req, res)) return
if (req.query.videoId && (req.query.videoChannelId || req.query.videoChannelName)) {
- return res.status(400).send({
- message: 'videoId cannot be mixed with a channel filter'
- }).end()
+ return res.fail({ message: 'videoId cannot be mixed with a channel filter' })
}
if (req.query.videoId && !await doesVideoExist(req.query.videoId, res)) return
feedsFormatValidator,
setFeedFormatContentType,
videoFeedsValidator,
- videoSubscriptonFeedsValidator,
+ videoSubscriptionFeedsValidator,
videoCommentsFeedsValidator
}