Add locked request url to logged warning when concurrency exceeded.

colinmollenhour · Dec 29, 2023 · 47a1df3 · 47a1df3 · hostep · Feb 1, 2024
1 parent 3d3e497
commit 47a1df3
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/Cm/RedisSession/Handler.php b/src/Cm/RedisSession/Handler.php
@@ -512,12 +512,12 @@ public function read($sessionId)
                     // Overloaded sessions get 503 errors
                     $this->_redis->hIncrBy($sessionId, 'wait', -1);
                     $this->_sessionWritten = true; // Prevent session from getting written
-                    $writes = $this->_redis->hGet($sessionId, 'writes');
+                    [$writes, $lockedRequestUrl] = $this->_redis->hMGet($sessionId, ['writes','req']);
                     $this->_log(
                         sprintf(
                             'Session concurrency exceeded for ID %s; displaying HTTP 503 (%s waiting, %s total '
-                            . 'requests)',
-                            $sessionId, $waiting, $writes
+                            . 'requests) - Locked URL: %s',
+                            $sessionId, $waiting, $writes, $lockedRequestUrl
                         ),
                         LoggerInterface::WARNING
                     );