Merge branch '6.1' into 6.2

* 6.1:
  Typos In Comments
  typo s/retrieveing/retrieving
  Typos on 6.0 branch
  remove duplicated catch block
  s/<\br>/<br>
  [HttpClient] Fix retrying requests when the content is used by the strategy

Author: fabpot

RetryableHttpClient.php

@@ -60,7 +60,7 @@ public function request(string $method, string $url, array $options = []): Respo
         return new AsyncResponse($this->client, $method, $url, $options, function (ChunkInterface $chunk, AsyncContext $context) use ($method, $url, $options, &$retryCount, &$content, &$firstChunk) {
             $exception = null;
             try {
-                if ($chunk->isTimeout() || null !== $chunk->getInformationalStatus() || $context->getInfo('canceled')) {
+                if ($context->getInfo('canceled') || $chunk->isTimeout() || null !== $chunk->getInformationalStatus()) {
                     yield $chunk;
 
                     return;
@@ -118,6 +118,8 @@ public function request(string $method, string $url, array $options = []): Respo
 
             $delay = $this->getDelayFromHeader($context->getHeaders()) ?? $this->strategy->getDelay($context, !$exception && $chunk->isLast() ? $content : null, $exception);
             ++$retryCount;
+            $content = '';
+            $firstChunk = null;
 
             $this->logger->info('Try #{count} after {delay}ms'.($exception ? ': '.$exception->getMessage() : ', status code: '.$context->getStatusCode()), [
                 'count' => $retryCount,

Tests/RetryableHttpClientTest.php

@@ -62,21 +62,22 @@ public function testRetryWithBody()
     {
         $client = new RetryableHttpClient(
             new MockHttpClient([
-                new MockResponse('', ['http_code' => 500]),
-                new MockResponse('', ['http_code' => 200]),
+                new MockResponse('abc', ['http_code' => 500]),
+                new MockResponse('def', ['http_code' => 200]),
             ]),
             new class(GenericRetryStrategy::DEFAULT_RETRY_STATUS_CODES, 0) extends GenericRetryStrategy {
                 public function shouldRetry(AsyncContext $context, ?string $responseContent, ?TransportExceptionInterface $exception): ?bool
                 {
-                    return null === $responseContent ? null : 200 !== $context->getStatusCode();
+                    return 500 === $context->getStatusCode() && null === $responseContent ? null : 200 !== $context->getStatusCode();
                 }
             },
-            1
+            2
         );
 
         $response = $client->request('GET', 'http://example.com/foo-bar');
 
         self::assertSame(200, $response->getStatusCode());
+        self::assertSame('def', $response->getContent());
     }
 
     public function testRetryWithBodyKeepContent()