diff options
-rw-r--r-- | server/controllers/plugins.ts | 3 | ||||
-rw-r--r-- | server/tests/fixtures/peertube-plugin-test-five/main.js | 2 | ||||
-rw-r--r-- | server/tests/plugins/plugin-router.ts | 21 |
3 files changed, 26 insertions, 0 deletions
diff --git a/server/controllers/plugins.ts b/server/controllers/plugins.ts index 18c6613e2..6a1ccc0bf 100644 --- a/server/controllers/plugins.ts +++ b/server/controllers/plugins.ts | |||
@@ -9,6 +9,7 @@ import { getCompleteLocale, is18nLocale } from '../../shared/core-utils/i18n' | |||
9 | import { PluginType } from '../../shared/models/plugins/plugin.type' | 9 | import { PluginType } from '../../shared/models/plugins/plugin.type' |
10 | import { isTestInstance } from '../helpers/core-utils' | 10 | import { isTestInstance } from '../helpers/core-utils' |
11 | import { logger } from '@server/helpers/logger' | 11 | import { logger } from '@server/helpers/logger' |
12 | import { optionalAuthenticate } from '@server/middlewares/oauth' | ||
12 | 13 | ||
13 | const sendFileOptions = { | 14 | const sendFileOptions = { |
14 | maxAge: '30 days', | 15 | maxAge: '30 days', |
@@ -45,11 +46,13 @@ pluginsRouter.get('/plugins/:pluginName/:pluginVersion/client-scripts/:staticEnd | |||
45 | 46 | ||
46 | pluginsRouter.use('/plugins/:pluginName/router', | 47 | pluginsRouter.use('/plugins/:pluginName/router', |
47 | getPluginValidator(PluginType.PLUGIN, false), | 48 | getPluginValidator(PluginType.PLUGIN, false), |
49 | optionalAuthenticate, | ||
48 | servePluginCustomRoutes | 50 | servePluginCustomRoutes |
49 | ) | 51 | ) |
50 | 52 | ||
51 | pluginsRouter.use('/plugins/:pluginName/:pluginVersion/router', | 53 | pluginsRouter.use('/plugins/:pluginName/:pluginVersion/router', |
52 | getPluginValidator(PluginType.PLUGIN), | 54 | getPluginValidator(PluginType.PLUGIN), |
55 | optionalAuthenticate, | ||
53 | servePluginCustomRoutes | 56 | servePluginCustomRoutes |
54 | ) | 57 | ) |
55 | 58 | ||
diff --git a/server/tests/fixtures/peertube-plugin-test-five/main.js b/server/tests/fixtures/peertube-plugin-test-five/main.js index c1435b928..07dd18654 100644 --- a/server/tests/fixtures/peertube-plugin-test-five/main.js +++ b/server/tests/fixtures/peertube-plugin-test-five/main.js | |||
@@ -4,6 +4,8 @@ async function register ({ | |||
4 | const router = getRouter() | 4 | const router = getRouter() |
5 | router.get('/ping', (req, res) => res.json({ message: 'pong' })) | 5 | router.get('/ping', (req, res) => res.json({ message: 'pong' })) |
6 | 6 | ||
7 | router.get('/is-authenticated', (req, res) => res.json({ isAuthenticated: res.locals.authenticated })) | ||
8 | |||
7 | router.post('/form/post/mirror', (req, res) => { | 9 | router.post('/form/post/mirror', (req, res) => { |
8 | res.json(req.body) | 10 | res.json(req.body) |
9 | }) | 11 | }) |
diff --git a/server/tests/plugins/plugin-router.ts b/server/tests/plugins/plugin-router.ts index 9e78568cd..5392acc51 100644 --- a/server/tests/plugins/plugin-router.ts +++ b/server/tests/plugins/plugin-router.ts | |||
@@ -44,6 +44,27 @@ describe('Test plugin helpers', function () { | |||
44 | } | 44 | } |
45 | }) | 45 | }) |
46 | 46 | ||
47 | it('Should check if authenticated', async function () { | ||
48 | for (const path of basePaths) { | ||
49 | const res = await makeGetRequest({ | ||
50 | url: server.url, | ||
51 | path: path + 'is-authenticated', | ||
52 | token: server.accessToken, | ||
53 | statusCodeExpected: 200 | ||
54 | }) | ||
55 | |||
56 | expect(res.body.isAuthenticated).to.equal(undefined) | ||
57 | |||
58 | const secRes = await makeGetRequest({ | ||
59 | url: server.url, | ||
60 | path: path + 'is-authenticated', | ||
61 | statusCodeExpected: 200 | ||
62 | }) | ||
63 | |||
64 | expect(secRes.body.isAuthenticated).to.equal(false) | ||
65 | } | ||
66 | }) | ||
67 | |||
47 | it('Should mirror post body', async function () { | 68 | it('Should mirror post body', async function () { |
48 | const body = { | 69 | const body = { |
49 | hello: 'world', | 70 | hello: 'world', |