[HttpClient] dont display any content when none has been collected

symfony · Feb 2, 2020 · 36536c9 · 36536c9
1 parent fb732df
commit 36536c9
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/src/Symfony/Component/HttpClient/DataCollector/HttpClientDataCollector.php b/src/Symfony/Component/HttpClient/DataCollector/HttpClientDataCollector.php
@@ -145,13 +145,15 @@ private function collectOnClient(TraceableHttpClient $client): array
                     $content = [$content];
                 }
 
-                $k = 'response_content';
+                $content = ['response_content' => $content];
+            } elseif (\is_array($content)) {
+                $content = ['response_json' => $content];
             } else {
-                $k = 'response_json';
+                $content = [];
             }
 
             $debugInfo = array_diff_key($info, $baseInfo);
-            $info = ['info' => $debugInfo] + array_diff_key($info, $debugInfo) + [$k => $content];
+            $info = ['info' => $debugInfo] + array_diff_key($info, $debugInfo) + $content;
             unset($traces[$i]['info']); // break PHP reference used by TraceableHttpClient
             $traces[$i]['info'] = $this->cloneVar($info);
             $traces[$i]['options'] = $this->cloneVar($trace['options']);

diff --git a/src/Symfony/Component/HttpClient/TraceableHttpClient.php b/src/Symfony/Component/HttpClient/TraceableHttpClient.php
@@ -37,7 +37,7 @@ public function __construct(HttpClientInterface $client)
      */
     public function request(string $method, string $url, array $options = []): ResponseInterface
     {
-        $content = '';
+        $content = null;
         $traceInfo = [];
         $this->tracedRequests[] = [
             'method' => $method,