diff options
author | Ronan <flyingrub@protonmail.com> | 2017-10-16 11:43:40 +0200 |
---|---|---|
committer | Bigard Florian <florian.bigard@gmail.com> | 2017-10-16 11:43:40 +0200 |
commit | 8376734ee3a58edc960cb41663de5cc52c760f5b (patch) | |
tree | bf9fcdd489a5f1800049ee7dd5478138de3b097a /server/middlewares/validators/videos.ts | |
parent | d8755eed1e452d2efbfc983af0e9d228d152bf6b (diff) | |
download | PeerTube-8376734ee3a58edc960cb41663de5cc52c760f5b.tar.gz PeerTube-8376734ee3a58edc960cb41663de5cc52c760f5b.tar.zst PeerTube-8376734ee3a58edc960cb41663de5cc52c760f5b.zip |
Handle express-validator error on the client side and fix #96 (#98)
* Handle express-validator error on the client side
* More meaningfull error for not supported format
Diffstat (limited to 'server/middlewares/validators/videos.ts')
-rw-r--r-- | server/middlewares/validators/videos.ts | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/server/middlewares/validators/videos.ts b/server/middlewares/validators/videos.ts index deed07524..f63348e66 100644 --- a/server/middlewares/validators/videos.ts +++ b/server/middlewares/validators/videos.ts | |||
@@ -23,7 +23,10 @@ import { | |||
23 | } from '../../helpers' | 23 | } from '../../helpers' |
24 | 24 | ||
25 | const videosAddValidator = [ | 25 | const videosAddValidator = [ |
26 | body('videofile').custom((value, { req }) => isVideoFile(req.files)).withMessage('Should have a valid file'), | 26 | body('videofile').custom((value, { req }) => isVideoFile(req.files)).withMessage( |
27 | 'This file is not supported. Are you sure it is of the following type : ' | ||
28 | + CONSTRAINTS_FIELDS.VIDEOS.EXTNAME | ||
29 | ), | ||
27 | body('name').custom(isVideoNameValid).withMessage('Should have a valid name'), | 30 | body('name').custom(isVideoNameValid).withMessage('Should have a valid name'), |
28 | body('category').custom(isVideoCategoryValid).withMessage('Should have a valid category'), | 31 | body('category').custom(isVideoCategoryValid).withMessage('Should have a valid category'), |
29 | body('licence').custom(isVideoLicenceValid).withMessage('Should have a valid licence'), | 32 | body('licence').custom(isVideoLicenceValid).withMessage('Should have a valid licence'), |