Browse Source

add trim length

zahmis 5 years ago
parent
commit
5110e95b16
1 changed files with 8 additions and 5 deletions
  1. 8 5
      src/server/routes/apiv3/slack-integration-settings.js

+ 8 - 5
src/server/routes/apiv3/slack-integration-settings.js

@@ -8,7 +8,6 @@ const loggerFactory = require('@alias/logger');
 const { getConnectionStatuses } = require('@growi/slack');
 
 const ErrorV3 = require('../../models/vo/error-apiv3');
-const apiv3FormValidator = require('../../middlewares/apiv3-form-validator');
 
 const logger = loggerFactory('growi:routes:apiv3:notification-setting');
 
@@ -60,9 +59,13 @@ module.exports = (crowi) => {
       body('channel').trim().not().isEmpty()
         .isString(),
     ],
-    Tokens: [
-      body('tokenGtoP').not().isEmpty().isString(),
-      body('tokenPtoG').not().isEmpty().isString(),
+    AccessTokens: [
+      body('tokenGtoP').trim().not().isEmpty()
+        .isString()
+        .isLength({ min: 1 }),
+      body('tokenPtoG').trim().not().isEmpty()
+        .isString()
+        .isLength({ min: 1 }),
     ],
   };
 
@@ -383,7 +386,7 @@ module.exports = (crowi) => {
    *          200:
    *            description: Succeeded to delete access tokens for slack
    */
-  router.delete('/slack-app-integration', loginRequiredStrictly, adminRequired, csrf, validator.Tokens, apiv3FormValidator, async(req, res) => {
+  router.delete('/slack-app-integration', validator.AccessTokens, apiV3FormValidator, async(req, res) => {
     const SlackAppIntegration = mongoose.model('SlackAppIntegration');
     const { tokenGtoP, tokenPtoG } = req.body;
     try {