Provisioning and LDAP

Hi,

automatic provisioning does not work with LDAP, as %USERID% fetches the UID, which is usually a UUID Value and completly senseless for Email, instead with LDAP we need the DisplayName (which is the real user login Name).

so please add the patch below to make also a %DISPLAYNAME% parameter available.

*** Config.php.orig     2020-01-27 22:04:10.591445448 +0100
--- Config.php  2020-01-27 22:05:04.002460570 +0100
***************
*** 112,118 ****
        }

        /**
!        * Replace %USERID% and %EMAIL% to allow special configurations
         *
         * @param string $original
         * @param IUser $user
--- 112,118 ----
        }

        /**
!        * Replace %USERID%, %DISPLAYNAME% and %EMAIL% to allow special configurations
         *
         * @param string $original
         * @param IUser $user
***************
*** 122,127 ****
--- 122,130 ----
                if ($user->getUID() !== null) {
                        $original = str_replace('%USERID%', $user->getUID(), $original);
                }
+                 if ($user->getDisplayName() !== null) {
+                         $original = str_replace('%DISPLAYNAME%', $user->getDisplayName(), $original);
+                 }
                if ($user->getEMailAddress() !== null) {
                        $original = str_replace('%EMAIL%', $user->getEMailAddress(), $original);
                }

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Review the parameter replacement method in Config.php and its existing %USERID% and %EMAIL% behavior. Add the requested %DISPLAYNAME% handling, update the nearby documentation comment, and verify that LDAP provisioning can use the new placeholder without changing existing substitutions.