Allow extra request data to be sent to the log

Author: ilijastuden

src/Logger.php

@@ -340,17 +340,25 @@ private function serializeException($argument_name, $exception, array &$context)
     /**
      * {@inheritdoc}
      */
-    public function requestSummary($exec_time_in_s, $memory_usage, $query_count, $query_exec_time)
+    public function requestSummary($exec_time_in_s, $memory_usage, $query_count, $query_exec_time, array $extra = [])
     {
         $event_name = $this->env_arguments['sapi'] == 'cli' ? 'cli_request' : 'http_request';
 
-        return $this->event($event_name, 'Request {signature} done in {exec_time} miliseconds', array_merge($this->request_summary_arguments, $this->response_summary_arguments, [
-            'signature' => $this->request_signature,
-            'exec_time' => $exec_time_in_s > 0 ? ceil($exec_time_in_s * 1000) : 0, // Log execution time in ms
-            'memory_usage' => $memory_usage,
-            'query_count' => $query_count,
-            'query_time' => $query_exec_time > 0 ? ceil($query_exec_time * 1000) : 0, // Log execution time in ms
-        ]));
+        return $this->event($event_name,
+            'Request {signature} done in {exec_time} miliseconds',
+            array_merge(
+                $this->request_summary_arguments,
+                $this->response_summary_arguments,
+                $extra,
+                [
+                    'signature' => $this->request_signature,
+                    'exec_time' => $exec_time_in_s > 0 ? ceil($exec_time_in_s * 1000) : 0, // Log execution time in ms
+                    'memory_usage' => $memory_usage,
+                    'query_count' => $query_count,
+                    'query_time' => $query_exec_time > 0 ? ceil($query_exec_time * 1000) : 0, // Log execution time in ms
+                ]
+            )
+        );
     }
 
     /**

src/LoggerInterface.php

@@ -125,8 +125,9 @@ public function flushBufferOnShutdown();
      * @param int   $memory_usage
      * @param int   $query_count
      * @param float $query_exec_time
+     * @param array $extra
      */
-    public function requestSummary($exec_time_in_s, $memory_usage, $query_count, $query_exec_time);
+    public function requestSummary($exec_time_in_s, $memory_usage, $query_count, $query_exec_time, array $extra = []);
 
     /**
      * Log an event (info with mandatory event attribute).

test/src/LoggerTest.php

@@ -440,7 +440,10 @@ public function testRequestSummary()
             0.356,
             1024 * 1024,
             15,
-            0.235
+            0.235,
+            [
+                'first' => 'second'
+            ]
         );
 
         $this->assertCount(1, $this->logger->getBuffer());
@@ -454,5 +457,7 @@ public function testRequestSummary()
         $this->assertEquals(1048576, $this->logger->getBuffer()[0]['context']['memory_usage']);
         $this->assertEquals(15, $this->logger->getBuffer()[0]['context']['query_count']);
         $this->assertEquals(235, $this->logger->getBuffer()[0]['context']['query_time']);
+        $this->assertArrayHasKey('first', $this->logger->getBuffer()[0]['context']);
+        $this->assertSame('second', $this->logger->getBuffer()[0]['context']['first']);
     }
 }