From 8367bda4454d07613126bbcc512f8fc568d79149 Mon Sep 17 00:00:00 2001
From: =?utf8?q?C=C3=B4me=20Chilliet?= <come.chilliet@nextcloud.com>
Date: Thu, 4 Nov 2021 12:30:38 +0100
Subject: [PATCH] Put back length check to have a clear error
MIME-Version: 1.0
Content-Type: text/plain; charset=utf8
Content-Transfer-Encoding: 8bit

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>
---
 apps/user_ldap/lib/Mapping/AbstractMapping.php | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/apps/user_ldap/lib/Mapping/AbstractMapping.php b/apps/user_ldap/lib/Mapping/AbstractMapping.php
index ce20983937a..966d930481c 100644
--- a/apps/user_ldap/lib/Mapping/AbstractMapping.php
+++ b/apps/user_ldap/lib/Mapping/AbstractMapping.php
@@ -356,6 +356,17 @@ abstract class AbstractMapping {
 	 * @return bool
 	 */
 	public function map($fdn, $name, $uuid) {
+		if (mb_strlen($fdn) > 4096) {
+			\OC::$server->getLogger()->error(
+				'Cannot map, because the DN exceeds 4096 characters: {dn}',
+				[
+					'app' => 'user_ldap',
+					'dn' => $fdn,
+				]
+			);
+			return false;
+		}
+
 		$row = [
 			'ldap_dn_hash' => $this->getDNHash($fdn),
 			'ldap_dn' => $fdn,
-- 
2.39.5