ensure to trigger resetting and setupping LdapStrategy by router

lib/models/config.js

@@ -251,10 +251,10 @@ module.exports = function(crowi) {
     return getValueForCrowiNS(config, key);
   };
 
-  configSchema.statics.isEnabledLdap = function(config)
+  configSchema.statics.isEnabledPassportLdap = function(config)
   {
-    // TODO impl
-    return false;
+    const key = 'security:passport-ldap:isEnabled';
+    return getValueForCrowiNS(config, key);
   };
 
   configSchema.statics.isUploadable = function(config)

lib/routes/admin.js

@@ -526,8 +526,14 @@ module.exports = function(crowi, app) {
     debug('form content', form);
     return saveSettingAsync(form)
       .then(() => {
-        // re-setup strategy (force: true)
-        crowi.passportService.setupLdapStrategy(true);
+        const config = crowi.getConfig();
+
+        // reset strategy
+        crowi.passportService.resetLdapStrategy();
+        // setup strategy
+        if (Config.isEnabledPassportLdap(config)) {
+          crowi.passportService.setupLdapStrategy(true);
+        }
         return;
       })
       .then(() => {