Fix #49: Fix JSON rendering of non-UTF-8 encoded string

Author: devanych

CHANGELOG.md

@@ -1,9 +1,8 @@
 # Yii Error Handler Change Log
 
-
 ## 2.0.1 under development
 
-- no changes in this release.
+- Bug #49: Fix JSON rendering of non-UTF-8 encoded string (devanych)
 
 ## 2.0.0 November 09, 2021
 

src/Renderer/JsonRenderer.php

@@ -33,6 +33,6 @@ public function renderVerbose(Throwable $t, ServerRequestInterface $request = nu
             'file' => $t->getFile(),
             'line' => $t->getLine(),
             'trace' => $t->getTrace(),
-        ], JSON_THROW_ON_ERROR | JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES));
+        ], JSON_THROW_ON_ERROR | JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_INVALID_UTF8_SUBSTITUTE));
     }
 }

tests/Renderer/JsonRendererTest.php

@@ -9,6 +9,8 @@
 use Yiisoft\ErrorHandler\Renderer\JsonRenderer;
 
 use function json_encode;
+use function md5;
+use function pack;
 
 final class JsonRendererTest extends TestCase
 {
@@ -36,4 +38,21 @@ public function testRenderVerbose(): void
 
         $this->assertSame($content, (string) $data);
     }
+
+    public function testRenderVerboseWithNotUtf8String(): void
+    {
+        $renderer = new JsonRenderer();
+        $throwable = new RuntimeException(pack('H*', md5('binary string')));
+        $data = $renderer->renderVerbose($throwable);
+        $content = json_encode([
+            'type' => RuntimeException::class,
+            'message' => $throwable->getMessage(),
+            'code' => $throwable->getCode(),
+            'file' => $throwable->getFile(),
+            'line' => $throwable->getLine(),
+            'trace' => $throwable->getTrace(),
+        ], JSON_PRETTY_PRINT | JSON_UNESCAPED_SLASHES | JSON_INVALID_UTF8_SUBSTITUTE);
+
+        $this->assertSame($content, (string) $data);
+    }
 }