Merge pull request #6564 from kenjis/remove-utf8_encode

kenjis · web-flow · commit e9d72ea387fb · 2022-09-23T07:29:54.000+09:00
refactor: replace utf8_encode() with mb_convert_encoding()
diff --git a/system/Log/Handlers/ChromeLoggerHandler.php b/system/Log/Handlers/ChromeLoggerHandler.php
@@ -157,7 +157,9 @@ public function sendLogs(?ResponseInterface &$response = null)
             $response = Services::response(null, true);
         }
 
-        $data = base64_encode(utf8_encode(json_encode($this->json)));
+        $data = base64_encode(
+            mb_convert_encoding(json_encode($this->json), 'UTF-8', mb_list_encodings())
+        );
 
         $response->setHeader($this->header, $data);
     }

Original file line number	Diff line number	Diff line change
`@@ -157,7 +157,9 @@ public function sendLogs(?ResponseInterface &$response = null)`
`157`	`157`	`$response = Services::response(null, true);`
`158`	`158`	`}`
`159`	`159`
`160`		`- $data = base64_encode(utf8_encode(json_encode($this->json)));`
	`160`	`+ $data = base64_encode(`
	`161`	`+ mb_convert_encoding(json_encode($this->json), 'UTF-8', mb_list_encodings())`
	`162`	`+ );`
`161`	`163`
`162`	`164`	`$response->setHeader($this->header, $data);`
`163`	`165`	`}`