Merge pull request #172 from rollbar/fixed/issue-171-log-context-passed-directly-to-telemetry-body

Fixed #171 Log context passed directly to telemetry body.

Author: danielmorell

src/TelemetryListener.php

@@ -105,10 +105,10 @@ protected function logMessageHandler(MessageLogged $message): void
             EventType::Log,
             // Telemetry does not support all PSR-3 or RFC-5424 levels, so we need to convert them.
             Telemeter::getLevelFromPsrLevel($message->level),
-            array_merge(
-                $message->context,
-                ['message' => $message->message],
-            ),
+            [
+                'message' => $message->message,
+                'context' => $message->context,
+            ],
         );
     }
 

tests/TelemetryListenerTest.php

@@ -49,7 +49,7 @@ public function testTelemetryCapturesLaravelLogs(): void
 
         self::assertSame(EventLevel::Debug, $lastItem->level);
         self::assertSame('telemetry test message', $lastItem->body->message);
-        self::assertSame(['foo' => 'bar'], $lastItem->body->extra);
+        self::assertSame(['context' => ['foo' => 'bar']], $lastItem->body->extra);
     }
 
     public function testTelemetryDoesNotCaptureLaravelLogsWhenDisabled(): void