fix(SentryTrap): enhance error logging during envelope parsing and improve JSON header parsing

Author: roxblnfk

src/Handler/Http/Middleware/SentryTrap.php

@@ -6,6 +6,7 @@
 
 use Buggregator\Trap\Handler\Http\Middleware;
 use Buggregator\Trap\Handler\Http\Middleware\SentryTrap\EnvelopeParser;
+use Buggregator\Trap\Logger;
 use Buggregator\Trap\Proto\Frame;
 use Nyholm\Psr7\Response;
 use Psr\Http\Message\ResponseInterface;
@@ -21,6 +22,10 @@ final class SentryTrap implements Middleware
 {
     private const MAX_BODY_SIZE = 2 * 1024 * 1024;
 
+    public function __construct(
+        private readonly ?Logger $logger = null,
+    ) {}
+
     public function handle(ServerRequestInterface $request, callable $next): ResponseInterface
     {
         try {
@@ -69,8 +74,12 @@ private function processEnvelope(ServerRequestInterface $request): ResponseInter
         $time = $request->getAttribute('begin_at');
         $time = $time instanceof \DateTimeImmutable ? $time : new \DateTimeImmutable();
 
-        $frame = EnvelopeParser::parse($request->getBody(), $time);
-        \Fiber::suspend($frame);
+        try {
+            $frame = EnvelopeParser::parse($request->getBody(), $time);
+            \Fiber::suspend($frame);
+        } catch (\Throwable $e) {
+            $this->logger?->info('Sentry envelope parsing failed: %s', $e->getMessage());
+        }
 
         return new Response(200);
     }

src/Handler/Http/Middleware/SentryTrap/EnvelopeParser.php

@@ -23,7 +23,7 @@ public static function parse(
         \DateTimeImmutable $time = new \DateTimeImmutable(),
     ): SentryEnvelope {
         // Parse headers
-        $headers = \json_decode(self::readLine($stream), true, 4, JSON_THROW_ON_ERROR);
+        $headers = \json_decode(self::readLine($stream), true, 32, JSON_THROW_ON_ERROR);
 
         // Parse items
         $items = [];

src/Sender/Console/Renderer/Sentry/Header.php

@@ -5,6 +5,7 @@
 namespace Buggregator\Trap\Sender\Console\Renderer\Sentry;
 
 use Buggregator\Trap\Sender\Console\Support\Common;
+use Buggregator\Trap\Support\Json;
 use Symfony\Component\Console\Output\OutputInterface;
 
 /**
@@ -34,7 +35,11 @@ public static function renderMessageHeader(OutputInterface $output, array $messa
             isset($context['runtime']) and $meta['Runtime'] = \implode(' ', (array) $context['runtime']);
             isset($context['os']) and $meta['OS'] = \implode(' ', (array) $context['os']);
         }
-        isset($message['sdk']) and $meta['SDK'] = \implode(' ', (array) $message['sdk']);
+        $headersList = \array_map(
+            fn(mixed $value): string => \is_scalar($value) ? (string) $value : Json::encode($value),
+            (array) $message['sdk'],
+        );
+        isset($message['sdk']) and $meta['SDK'] = \implode(' ', $headersList);
 
         Common::renderMetadata($output, $meta);