Don't limit browser_session.user_agent to 4096 chars

Icinga · Jul 15, 2024 · 709b1d1 · 709b1d1
1 parent 8791c69
commit 709b1d1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/library/Notifications/Daemon/Server.php b/library/Notifications/Daemon/Server.php
@@ -406,8 +406,7 @@ protected function authenticate(Connection $connection, array $cookies, array $h
 
             if ($browserSession !== null) {
                 if (isset($headers['User-Agent'][0])) {
-                    // limit user-agent to 4k chars
-                    $userAgent = substr(trim($headers['User-Agent'][0]), 0, 4096);
+                    $userAgent = trim($headers['User-Agent'][0]);
                 } else {
                     $userAgent = 'default';
                 }

diff --git a/library/Notifications/ProvidedHook/SessionStorage.php b/library/Notifications/ProvidedHook/SessionStorage.php
@@ -39,8 +39,7 @@ public function onLogin(User $user): void
             // user successfully authenticated
             $rawUserAgent = (new UserAgent())->getAgent();
             if ($rawUserAgent) {
-                // limit user-agent to 4k chars
-                $userAgent = substr(trim($rawUserAgent), 0, 4096);
+                $userAgent = trim($rawUserAgent);
             } else {
                 $userAgent = 'default';
             }
@@ -124,8 +123,7 @@ public function onLogout(User $user): void
 
             $rawUserAgent = (new UserAgent())->getAgent();
             if ($rawUserAgent) {
-                // limit user-agent to 4k chars
-                $userAgent = substr(trim($rawUserAgent), 0, 4096);
+                $userAgent = trim($rawUserAgent);
             } else {
                 $userAgent = 'default';
             }