diff options
-rw-r--r-- | server/controllers/api/index.js | 8 | ||||
-rw-r--r-- | server/controllers/api/remote/index.js | 16 | ||||
-rw-r--r-- | server/controllers/api/remote/videos.js (renamed from server/controllers/api/remote.js) | 8 | ||||
-rw-r--r-- | server/helpers/utils.js | 5 |
4 files changed, 28 insertions, 9 deletions
diff --git a/server/controllers/api/index.js b/server/controllers/api/index.js index 4cb65ed55..f13ff922c 100644 --- a/server/controllers/api/index.js +++ b/server/controllers/api/index.js | |||
@@ -2,6 +2,8 @@ | |||
2 | 2 | ||
3 | const express = require('express') | 3 | const express = require('express') |
4 | 4 | ||
5 | const utils = require('../../helpers/utils') | ||
6 | |||
5 | const router = express.Router() | 7 | const router = express.Router() |
6 | 8 | ||
7 | const clientsController = require('./clients') | 9 | const clientsController = require('./clients') |
@@ -18,7 +20,7 @@ router.use('/requests', requestsController) | |||
18 | router.use('/users', usersController) | 20 | router.use('/users', usersController) |
19 | router.use('/videos', videosController) | 21 | router.use('/videos', videosController) |
20 | router.use('/ping', pong) | 22 | router.use('/ping', pong) |
21 | router.use('/*', badRequest) | 23 | router.use('/*', utils.badRequest) |
22 | 24 | ||
23 | // --------------------------------------------------------------------------- | 25 | // --------------------------------------------------------------------------- |
24 | 26 | ||
@@ -29,7 +31,3 @@ module.exports = router | |||
29 | function pong (req, res, next) { | 31 | function pong (req, res, next) { |
30 | return res.send('pong').status(200).end() | 32 | return res.send('pong').status(200).end() |
31 | } | 33 | } |
32 | |||
33 | function badRequest (req, res, next) { | ||
34 | res.type('json').status(400).end() | ||
35 | } | ||
diff --git a/server/controllers/api/remote/index.js b/server/controllers/api/remote/index.js new file mode 100644 index 000000000..2947632d5 --- /dev/null +++ b/server/controllers/api/remote/index.js | |||
@@ -0,0 +1,16 @@ | |||
1 | 'use strict' | ||
2 | |||
3 | const express = require('express') | ||
4 | |||
5 | const utils = require('../../../helpers/utils') | ||
6 | |||
7 | const router = express.Router() | ||
8 | |||
9 | const videosRemoteController = require('./videos') | ||
10 | |||
11 | router.use('/videos', videosRemoteController) | ||
12 | router.use('/*', utils.badRequest) | ||
13 | |||
14 | // --------------------------------------------------------------------------- | ||
15 | |||
16 | module.exports = router | ||
diff --git a/server/controllers/api/remote.js b/server/controllers/api/remote/videos.js index be5e6dc98..87c49bff9 100644 --- a/server/controllers/api/remote.js +++ b/server/controllers/api/remote/videos.js | |||
@@ -4,15 +4,15 @@ const eachSeries = require('async/eachSeries') | |||
4 | const express = require('express') | 4 | const express = require('express') |
5 | const waterfall = require('async/waterfall') | 5 | const waterfall = require('async/waterfall') |
6 | 6 | ||
7 | const db = require('../../initializers/database') | 7 | const db = require('../../../initializers/database') |
8 | const middlewares = require('../../middlewares') | 8 | const middlewares = require('../../../middlewares') |
9 | const secureMiddleware = middlewares.secure | 9 | const secureMiddleware = middlewares.secure |
10 | const validators = middlewares.validators.remote | 10 | const validators = middlewares.validators.remote |
11 | const logger = require('../../helpers/logger') | 11 | const logger = require('../../../helpers/logger') |
12 | 12 | ||
13 | const router = express.Router() | 13 | const router = express.Router() |
14 | 14 | ||
15 | router.post('/videos', | 15 | router.post('/', |
16 | validators.signature, | 16 | validators.signature, |
17 | secureMiddleware.checkSignature, | 17 | secureMiddleware.checkSignature, |
18 | validators.remoteVideos, | 18 | validators.remoteVideos, |
diff --git a/server/helpers/utils.js b/server/helpers/utils.js index 9f27671b6..7e0c9823c 100644 --- a/server/helpers/utils.js +++ b/server/helpers/utils.js | |||
@@ -5,11 +5,16 @@ const crypto = require('crypto') | |||
5 | const logger = require('./logger') | 5 | const logger = require('./logger') |
6 | 6 | ||
7 | const utils = { | 7 | const utils = { |
8 | badRequest, | ||
8 | cleanForExit, | 9 | cleanForExit, |
9 | generateRandomString, | 10 | generateRandomString, |
10 | isTestInstance | 11 | isTestInstance |
11 | } | 12 | } |
12 | 13 | ||
14 | function badRequest (req, res, next) { | ||
15 | res.type('json').status(400).end() | ||
16 | } | ||
17 | |||
13 | function generateRandomString (size, callback) { | 18 | function generateRandomString (size, callback) { |
14 | crypto.pseudoRandomBytes(size, function (err, raw) { | 19 | crypto.pseudoRandomBytes(size, function (err, raw) { |
15 | if (err) return callback(err) | 20 | if (err) return callback(err) |