X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=server%2Fhelpers%2Fcustom-validators%2Fmisc.ts;h=c19a3e5eba2692e7f1e87075d20509ede9bba365;hb=795212f7acc690c88c86d0fab8772f6564d59cb8;hp=3a3deab0c8f7b152ac56d08348d003b84286c247;hpb=97567dd81f508dd6295ac4d73d849aa2ce0a6549;p=github%2FChocobozzz%2FPeerTube.git diff --git a/server/helpers/custom-validators/misc.ts b/server/helpers/custom-validators/misc.ts index 3a3deab0c..c19a3e5eb 100644 --- a/server/helpers/custom-validators/misc.ts +++ b/server/helpers/custom-validators/misc.ts @@ -1,11 +1,21 @@ import 'multer' -import * as validator from 'validator' +import { UploadFilesForCheck } from 'express' +import { sep } from 'path' +import validator from 'validator' +import { isShortUUID, shortToUUID } from '../uuid' function exists (value: any) { return value !== undefined && value !== null } -function isArray (value: any) { +function isSafePath (p: string) { + return exists(p) && + (p + '').split(sep).every(part => { + return [ '..' ].includes(part) === false + }) +} + +function isArray (value: any): value is any[] { return Array.isArray(value) } @@ -13,6 +23,10 @@ function isNotEmptyIntArray (value: any) { return Array.isArray(value) && value.every(v => validator.isInt('' + v)) && value.length !== 0 } +function isNotEmptyStringArray (value: any) { + return Array.isArray(value) && value.every(v => typeof v === 'string' && v.length !== 0) && value.length !== 0 +} + function isArrayOf (value: any, validator: (value: any) => boolean) { return isArray(value) && value.every(v => validator(v)) } @@ -29,6 +43,10 @@ function isUUIDValid (value: string) { return exists(value) && validator.isUUID('' + value, 4) } +function areUUIDsValid (values: string[]) { + return isArray(values) && values.every(v => isUUIDValid(v)) +} + function isIdOrUUIDValid (value: string) { return isIdValid(value) || isUUIDValid(value) } @@ -37,29 +55,54 @@ function isBooleanValid (value: any) { return typeof value === 'boolean' || (typeof value === 'string' && validator.isBoolean(value)) } -function toIntOrNull (value: string) { - if (value === 'null') return null - - return validator.toInt(value) +function isIntOrNull (value: any) { + return value === null || validator.isInt('' + value) } -function toValueOrNull (value: string) { - if (value === 'null') return null +// --------------------------------------------------------------------------- - return value -} +function isFileFieldValid ( + files: { [ fieldname: string ]: Express.Multer.File[] } | Express.Multer.File[], + field: string, + optional = false +) { + // Should have files + if (!files) return optional + if (isArray(files)) return optional -function toArray (value: any) { - if (value && isArray(value) === false) return [ value ] + // Should have a file + const fileArray = files[field] + if (!fileArray || fileArray.length === 0) { + return optional + } - return value + // The file should exist + const file = fileArray[0] + if (!file || !file.originalname) return false + return file } -function toIntArray (value: any) { - if (!value) return [] - if (isArray(value) === false) return [ validator.toInt(value) ] +function isFileMimeTypeValid ( + files: UploadFilesForCheck, + mimeTypeRegex: string, + field: string, + optional = false +) { + // Should have files + if (!files) return optional + if (isArray(files)) return optional - return value.map(v => validator.toInt(v)) + // Should have a file + const fileArray = files[field] + if (!fileArray || fileArray.length === 0) { + return optional + } + + // The file should exist + const file = fileArray[0] + if (!file || !file.originalname) return false + + return new RegExp(`^${mimeTypeRegex}$`, 'i').test(file.mimetype) } function isFileValid ( @@ -74,13 +117,13 @@ function isFileValid ( if (isArray(files)) return optional // Should have a file - const fileArray = files[ field ] + const fileArray = files[field] if (!fileArray || fileArray.length === 0) { return optional } // The file should exist - const file = fileArray[ 0 ] + const file = fileArray[0] if (!file || !file.originalname) return false // Check size @@ -91,19 +134,77 @@ function isFileValid ( // --------------------------------------------------------------------------- +function toCompleteUUID (value: string) { + if (isShortUUID(value)) return shortToUUID(value) + + return value +} + +function toCompleteUUIDs (values: string[]) { + return values.map(v => toCompleteUUID(v)) +} + +function toIntOrNull (value: string) { + const v = toValueOrNull(value) + + if (v === null || v === undefined) return v + if (typeof v === 'number') return v + + return validator.toInt('' + v) +} + +function toBooleanOrNull (value: any) { + const v = toValueOrNull(value) + + if (v === null || v === undefined) return v + if (typeof v === 'boolean') return v + + return validator.toBoolean('' + v) +} + +function toValueOrNull (value: string) { + if (value === 'null') return null + + return value +} + +function toArray (value: any) { + if (value && isArray(value) === false) return [ value ] + + return value +} + +function toIntArray (value: any) { + if (!value) return [] + if (isArray(value) === false) return [ validator.toInt(value) ] + + return value.map(v => validator.toInt(v)) +} + +// --------------------------------------------------------------------------- + export { exists, isArrayOf, isNotEmptyIntArray, isArray, + isIntOrNull, isIdValid, + isSafePath, + isNotEmptyStringArray, isUUIDValid, + toCompleteUUIDs, + toCompleteUUID, isIdOrUUIDValid, isDateValid, toValueOrNull, + toBooleanOrNull, isBooleanValid, toIntOrNull, + areUUIDsValid, toArray, toIntArray, + isFileFieldValid, + isFileMimeTypeValid, isFileValid }