remove unnecessary code

packages/app/src/lib/util/isSecurityEnv.js

@@ -1,12 +0,0 @@
-/**
- * return whether env belongs to Security settings
- * @param {string} key ex. 'security:passport-saml:isEnabled' is true
- * @returns {boolean}
- * @memberof envUtils
- */
-const isSecurityEnv = (key) => {
-  const array = key.split(':');
-  return (array[0] === 'security');
-};
-
-module.exports = isSecurityEnv;

packages/app/src/server/crowi/dev.js

@@ -97,33 +97,6 @@ class CrowiDev {
     this.setupNextjsStackFrame(app);
   }
 
-  // setupHeaderDebugger(app) {
-  //   logger.debug('setupHeaderDebugger');
-
-  //   app.use((req, res, next) => {
-  //     onHeaders(res, () => {
-  //       logger.debug('HEADERS GOING TO BE WRITTEN');
-  //     });
-  //     next();
-  //   });
-  // }
-
-  // setupBrowserSync(app) {
-  //   logger.debug('setupBrowserSync');
-
-  //   const browserSync = require('browser-sync');
-  //   const bs = browserSync.create().init({
-  //     logSnippet: false,
-  //     notify: false,
-  //     files: [
-  //       `${this.crowi.viewsDir}/**/*.html`,
-  //       `${this.crowi.publicDir}/**/*.js`,
-  //       `${this.crowi.publicDir}/**/*.css`,
-  //     ],
-  //   });
-  //   app.use(require('connect-browser-sync')(bs));
-  // }
-
   setupNextBundleAnalyzer(app) {
     const next = nextFactory(this.crowi);
     app.use('/analyze', express.static(path.resolve(__dirname, '../../../.next/analyze')));

packages/app/src/server/crowi/express-init.js

@@ -20,10 +20,6 @@ module.exports = function(crowi, app) {
   const expressSession = require('express-session');
   const flash = require('connect-flash');
   const mongoSanitize = require('express-mongo-sanitize');
-  // const i18next = require('i18next');
-  // const i18nFsBackend = require('i18next-node-fs-backend');
-  // const i18nSprintf = require('i18next-sprintf-postprocessor');
-  // const i18nMiddleware = require('i18next-express-middleware');
 
   const promster = require('../middlewares/promster')(crowi, app);
   const registerSafeRedirect = require('../middlewares/safe-redirect')();
@@ -31,33 +27,9 @@ module.exports = function(crowi, app) {
   const autoReconnectToS2sMsgServer = require('../middlewares/auto-reconnect-to-s2s-msg-server')(crowi);
 
   const avoidSessionRoutes = require('../routes/avoid-session-routes');
-  // const i18nUserSettingDetector = require('../util/i18nUserSettingDetector');
 
   const env = crowi.node_env;
 
-  // const lngDetector = new i18nMiddleware.LanguageDetector();
-  // lngDetector.addDetector(i18nUserSettingDetector);
-
-  // i18next
-  //   .use(lngDetector)
-  //   .use(i18nFsBackend)
-  //   .use(i18nSprintf)
-  //   .init({
-  //     // debug: true,
-  //     fallbackLng: ['en_US'],
-  //     whitelist: i18n.locales,
-  //     backend: {
-  //       loadPath: `${localePath}/{{lng}}/translation.json`,
-  //     },
-  //     detection: {
-  //       order: ['userSettingDetector', 'header', 'navigator'],
-  //     },
-  //     overloadTranslationOptionHandler: i18nSprintf.overloadTranslationOptionHandler,
-
-  //     // change nsSeparator from ':' to '::' because ':' is used in config keys and these are used in i18n keys
-  //     nsSeparator: '::',
-  //   });
-
   app.use(compression());
 
 
@@ -165,7 +137,6 @@ module.exports = function(crowi, app) {
   app.use(injectCurrentuserToLocalvars);
   app.use(autoReconnectToS2sMsgServer);
 
-  // app.use(i18nMiddleware.handle(i18next));
   // TODO: Remove this workaround implementation when i18n works correctly.
   //       For now, req.t returns string given to req.t(string)
   app.use((req, res, next) => {

packages/app/src/server/util/i18nUserSettingDetector.js

@@ -1,15 +0,0 @@
-module.exports = {
-  name: 'userSettingDetector',
-
-  lookup(req, res, options) {
-    if (req.user == null) {
-      return null;
-    }
-
-    return req.user.lang || null;
-  },
-
-  cacheUserlanguage(req, res, lng, options) {
-    // nothing to do
-  },
-};