Moved link-sharing-required middleware to inside share-links

src/server/middlewares/link-sharing-required.js

@@ -1,14 +0,0 @@
-const loggerFactory = require('@alias/logger');
-const ErrorV3 = require('../models/vo/error-apiv3');
-
-const logger = loggerFactory('growi:middleware:link-sharing-required');
-
-module.exports = crowi => (req, res, next) => {
-  const isLinkSharingDisabled = crowi.configManager.getConfig('crowi', 'security:disableLinkSharing');
-  logger.debug(`isLinkSharingDisabled: ${isLinkSharingDisabled}`);
-
-  if (isLinkSharingDisabled) {
-    return res.apiv3Err(new ErrorV3('Link sharing is disabled', 'link-sharing-disabled'));
-  }
-  next();
-};

src/server/routes/apiv3/share-links.js

@@ -25,12 +25,24 @@ const today = new Date();
 module.exports = (crowi) => {
   const loginRequired = require('../../middlewares/login-required')(crowi);
   const adminRequired = require('../../middlewares/admin-required')(crowi);
-  const linkSharingRequired = require('../../middlewares/link-sharing-required')(crowi);
   const csrf = require('../../middlewares/csrf')(crowi);
   const apiV3FormValidator = require('../../middlewares/apiv3-form-validator')(crowi);
   const ShareLink = crowi.model('ShareLink');
   const Page = crowi.model('Page');
 
+  /**
+   * middleware to limit link sharing
+   */
+  const linkSharingRequired = (req, res, next) => {
+    const isLinkSharingDisabled = crowi.configManager.getConfig('crowi', 'security:disableLinkSharing');
+    logger.debug(`isLinkSharingDisabled: ${isLinkSharingDisabled}`);
+
+    if (isLinkSharingDisabled) {
+      return res.apiv3Err(new ErrorV3('Link sharing is disabled', 'link-sharing-disabled'));
+    }
+    next();
+  };
+
   validator.getShareLinks = [
     // validate the page id is MongoId
     query('relatedPage').isMongoId().withMessage('Page Id is required'),