LDAP Driver: add a parameter for the users count cache TTL

Author: cdujeu

core/src/plugins/auth.ldap/class.ldapAuthDriver.php

@@ -890,10 +890,12 @@ public function fakeMemberOf($conn, $groupDN, $filterString, $atts, &$entry)
 
     public function getCountFromCache()
     {
+        $ttl = $this->getOption("LDAP_COUNT_CACHE_TTL");
+        if(empty($ttl)) $ttl = 1;
         $fileName = "ldap.ser";
         if (file_exists($this->getPluginCacheDir() . DIRECTORY_SEPARATOR . $fileName)) {
             $fileContent = unserialize(file_get_contents($this->getPluginCacheDir() . DIRECTORY_SEPARATOR . $fileName));
-            if (($fileContent) && ($fileContent["count"]) && ($fileContent["timestamp"]) && ((time() - $fileContent["timestamp"]) < 60 * 30)) {
+            if (($fileContent) && ($fileContent["count"]) && ($fileContent["timestamp"]) && ((time() - $fileContent["timestamp"]) < 60 * 60 * $ttl)) {
                 return $fileContent;
             }
         }

core/src/plugins/auth.ldap/manifest.xml

@@ -28,6 +28,7 @@
         <param name="MAPPING_LDAP_PARAM" type="string" replicationGroup="MAPPING" label="CONF_MESSAGE[LDAP attribute]" description="CONF_MESSAGE[Name of the LDAP attribute to read]" mandatory="false"/>
         <param name="MAPPING_LOCAL_TYPE" type="select" replicationGroup="MAPPING" choices="plugin_param|Plugin Parameter,role_id|Role Id,group_path|Group Path,profile|Profile" label="CONF_MESSAGE[Mapping Type]" description="CONF_MESSAGE[Determine the type of mapping]" mandatory="false"/>
         <param name="MAPPING_LOCAL_PARAM" type="string" replicationGroup="MAPPING" label="CONF_MESSAGE[Plugin parameter]" description="CONF_MESSAGE[Name of the custom local parameter to set]" mandatory="false"/>
+        <param type="integer" name="LDAP_COUNT_CACHE_TTL" label="CONF_MESSAGE[Cache User Count (hours)]" description="CONF_MESSAGE[Locally cache the total number of users during X hours. Can be handy for huge directories.]" mandatory="false" default="1"/>
         <param type="string" name="TEST_USER" label="CONF_MESSAGE[Test User]" description="CONF_MESSAGE[Use the Test Connexion button to check if this user is correctly found in your LDAP directory.]" mandatory="false"/>
         <param type="button" name="TEST_LDAP" choices="run_plugin_action:auth.ldap:testLDAPConnexion" label="CONF_MESSAGE[Test Connexion]" description="CONF_MESSAGE[Try to connect to LDAP]" mandatory="false"/>
     </server_settings>