Просмотр исходного кода

set correct middlewares to attachment-refs api endpoint

Yuki Takei 4 лет назад
Родитель
Сommit
8df9a9f02f
1 измененных файлов с 12 добавлено и 2 удалено
  1. 12 2
      packages/plugin-attachment-refs/src/server/routes/refs.js

+ 12 - 2
packages/plugin-attachment-refs/src/server/routes/refs.js

@@ -6,9 +6,19 @@ const { OptionParser } = customTagUtils;
 
 
 const logger = loggerFactory('growi-plugin:attachment-refs:routes:refs');
 const logger = loggerFactory('growi-plugin:attachment-refs:routes:refs');
 
 
+
+const loginRequiredFallback = (req, res) => {
+  return res.status(403).send('login required');
+};
+
+
 module.exports = (crowi) => {
 module.exports = (crowi) => {
   const express = crowi.require('express');
   const express = crowi.require('express');
   const mongoose = crowi.require('mongoose');
   const mongoose = crowi.require('mongoose');
+
+  const loginRequired = crowi.require('../middlewares/login-required')(crowi, false, loginRequiredFallback);
+  const accessTokenParser = crowi.require('../middlewares/access-token-parser')(crowi);
+
   const router = express.Router();
   const router = express.Router();
 
 
   const ObjectId = mongoose.Types.ObjectId;
   const ObjectId = mongoose.Types.ObjectId;
@@ -68,7 +78,7 @@ module.exports = (crowi) => {
   /**
   /**
    * return an Attachment model
    * return an Attachment model
    */
    */
-  router.get('/ref', async(req, res) => {
+  router.get('/ref', accessTokenParser, loginRequired, async(req, res) => {
     const user = req.user;
     const user = req.user;
     const { pagePath, fileNameOrId } = req.query;
     const { pagePath, fileNameOrId } = req.query;
     // eslint-disable-next-line no-unused-vars
     // eslint-disable-next-line no-unused-vars
@@ -128,7 +138,7 @@ module.exports = (crowi) => {
   /**
   /**
    * return a list of Attachment
    * return a list of Attachment
    */
    */
-  router.get('/refs', async(req, res) => {
+  router.get('/refs', accessTokenParser, loginRequired, async(req, res) => {
     const user = req.user;
     const user = req.user;
     const { prefix, pagePath } = req.query;
     const { prefix, pagePath } = req.query;
     const options = JSON.parse(req.query.options);
     const options = JSON.parse(req.query.options);