Log a warning when a user or group is mapped with a suffix

Most of the time, this is the result of a misconfiguration by the admin
 and is not what is expected.

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

diff --git a/apps/user_ldap/lib/Access.php b/apps/user_ldap/lib/Access.php
index 1cc0c62ff1d177353c542c7184173a5eed496887..d3510e7a398f0b282b66c8113d54ecd9efb033b7 100644 (file)
--- a/apps/user_ldap/lib/Access.php
+++ b/apps/user_ldap/lib/Access.php
@@ -589,6 +589,15 @@ class Access extends LDAPUtility {
                $altName = $this->createAltInternalOwnCloudName($intName, $isUser);
                if (is_string($altName)) {
                        if ($this->mapAndAnnounceIfApplicable($mapper, $fdn, $altName, $uuid, $isUser)) {
+                               $this->logger->warning(
+                                       'Mapped {fdn} as {altName} because of a name collision on {intName}.',
+                                       [
+                                               'fdn' => $fdn,
+                                               'altName' => $altName,
+                                               'intName' => $intName,
+                                               'app' => 'user_ldap',
+                                       ]
+                               );
                                $newlyMapped = true;
                                return $altName;
                        }