From: Hypolite Petovan <hypolite@mrpetovan.com>
Date: Sun, 21 Oct 2018 21:28:40 +0000 (-0400)
Subject: Sets maximum to 64 for username_min/max_length config keys
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=4ec56a097522fb16f5389c6a2df5a06ae250c23f;p=friendica.git

Sets maximum to 64 for username_min/max_length config keys
---

diff --git a/config/config.ini.php b/config/config.ini.php
index 8d1c66dbb5..fc9855cd69 100644
--- a/config/config.ini.php
+++ b/config/config.ini.php
@@ -367,13 +367,13 @@ urlpath =
 ; username_min_length (Integer)
 ; The minimum character length a username can be.
 ; This length is check once the username has been trimmed and multiple spaces have been collapsed into one.
-; Minimum for this config value is 1. Maximum is 150 as the resulting profile URL shouldn't be longer than 255 chars.
+; Minimum for this config value is 1. Maximum is 64 as the resulting profile URL mustn't be longer than 255 chars.
 username_min_length = 3
 
 ; username_max_length (Integer)
 ; The maximum character length a username can be.
 ; This length is check once the username has been trimmed and multiple spaces have been collapsed into one.
-; Minimum for this config value is 1. Maximum is 150 as the resulting profile URL shouldn't be longer than 255 chars.
+; Minimum for this config value is 1. Maximum is 64 as the resulting profile URL mustn't be longer than 255 chars.
 username_max_length = 48
 
 ; worker_cooldown (Integer)
diff --git a/src/Model/User.php b/src/Model/User.php
index 4dcd507048..bca3e73f5d 100644
--- a/src/Model/User.php
+++ b/src/Model/User.php
@@ -466,8 +466,8 @@ class User
 		// collapse multiple spaces in name
 		$username = preg_replace('/ +/', ' ', $username);
 
-		$username_min_length = max(1, min(150, intval(Config::get('system', 'username_min_length', 3))));
-		$username_max_length = max(1, min(150, intval(Config::get('system', 'username_max_length', 48))));
+		$username_min_length = max(1, min(64, intval(Config::get('system', 'username_min_length', 3))));
+		$username_max_length = max(1, min(64, intval(Config::get('system', 'username_max_length', 48))));
 
 		if ($username_min_length > $username_max_length) {
 			logger(L10n::t('system.username_min_length (%s) and system.username_max_length (%s) are excluding each other, swapping values.', $username_min_length, $username_max_length), LOGGER_WARNING);