bug symfony#31069 Make sure that logged URL is the same as the one which is requested. (drupol)

This PR was squashed before being merged into the 4.3-dev branch (closes symfony#31069). Discussion ---------- Make sure that logged URL is the same as the one which is requested. | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony#31068 | License | MIT Commits ------- 526cad6 Make sure that logged URL is the same as the one which is requested.

Author: nicolas-grekas

src/Symfony/Component/HttpClient/NativeHttpClient.php

@@ -159,6 +159,8 @@ public function request(string $method, string $url, array $options = []): Respo
  $this->multi->dnsCache = $options['resolve'] + $this->multi->dnsCache;
  }
 
+ $this->logger && $this->logger->info(sprintf('Request: %s %s', $method, implode('', $url)));
+
  [$host, $port, $url['authority']] = self::dnsResolve($url, $this->multi, $info, $onProgress);
 
  if (!isset($options['headers']['host'])) {
@@ -208,10 +210,7 @@ public function request(string $method, string $url, array $options = []): Respo
  $context = stream_context_create($context, ['notification' => $notification]);
  self::configureHeadersAndProxy($context, $host, $options['request_headers'], $proxy, $noProxy);
 
- $url = implode('', $url);
- $this->logger && $this->logger->info(sprintf('Request: %s %s', $method, $url));
-
- return new NativeResponse($this->multi, $context, $url, $options, $gzipEnabled, $info, $resolveRedirect, $onProgress, $this->logger);
+ return new NativeResponse($this->multi, $context, implode('', $url), $options, $gzipEnabled, $info, $resolveRedirect, $onProgress, $this->logger);
  }
 
  /**