Logger - add headers into log file

Ecomailcz · Oct 13, 2020 · 93218b0 · 93218b0
1 parent c76f3ab
commit 93218b0
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/src/Http/HttpClient.php b/src/Http/HttpClient.php
@@ -13,6 +13,7 @@
 use function curl_getinfo;
 use function date;
 use function dirname;
+use function implode;
 use function is_string;
 use function mb_strlen;
 use function microtime;
@@ -73,15 +74,22 @@ public function request(
         $statusCode = curl_getinfo($ch, CURLINFO_HTTP_CODE);
 
         if ($config->getLogFilePath() !== null) {
+            $headersContents = [];
+
+            foreach ($headers as $key => $value) {
+                $headersContents[] = sprintf('%s:%s', $key, $value);
+            }
+
             $rootDir = dirname($config->getLogFilePath());
             $fileSystem = new Filesystem(new Local($rootDir, FILE_APPEND));
             $logContent = sprintf(
-                '%s METHOD: %s URL:%s TIME:%s STATUS:%s',
+                '%s METHOD: %s URL:%s TIME:%s STATUS:%s HEADERS: %s',
                 date('Y-m-d H:i:s'),
                 $httpMethod->getValue(),
                 $url,
                 number_format($responseTime, 2),
                 $statusCode,
+                implode(',', $headersContents),
             );
 
             if ($errorMessage !== null) {

diff --git a/tests/ClientTest.php b/tests/ClientTest.php
@@ -346,6 +346,7 @@ public function testLogRequest(): void
 
         Assert::assertTrue(file_exists($logPath));
         Assert::assertNotEmpty(file_get_contents($logPath));
+        unlink($logPath);
     }
 
     private function checkResponseStructure(EvidenceResult $result): void

diff --git a/tests/logs/log.txt b/tests/logs/log.txt