[HttpClient] Fix computing retry delay when using RetryableHttpClient

symfony · Sep 8, 2022 · 5fee724 · 5fee724
1 parent 1c6db72
commit 5fee724
Show file tree

Hide file tree

Showing 2 changed files with 39 additions and 1 deletion.
diff --git a/RetryableHttpClient.php b/RetryableHttpClient.php
@@ -138,7 +138,7 @@ private function getDelayFromHeader(array $headers): ?int
     {
         if (null !== $after = $headers['retry-after'][0] ?? null) {
             if (is_numeric($after)) {
-                return (int) $after * 1000;
+                return (int) ($after * 1000);
             }
 
             if (false !== $time = strtotime($after)) {

diff --git a/Tests/RetryableHttpClientTest.php b/Tests/RetryableHttpClientTest.php
@@ -187,4 +187,42 @@ public function testCancelOnTimeout()
             $response->cancel();
         }
     }
+
+    public function testRetryWithDelay()
+    {
+        $retryAfter = '0.46';
+
+        $client = new RetryableHttpClient(
+            new MockHttpClient([
+                new MockResponse('', [
+                    'http_code' => 503,
+                    'response_headers' => [
+                        'retry-after' => $retryAfter,
+                    ],
+                ]),
+                new MockResponse('', [
+                    'http_code' => 200,
+                ]),
+            ]),
+            new GenericRetryStrategy(),
+            1,
+            $logger = new class() extends TestLogger {
+                public array $context = [];
+
+                public function log($level, $message, array $context = []): void
+                {
+                    $this->context = $context;
+                    parent::log($level, $message, $context);
+                }
+            },
+        );
+
+        $client->request('GET', 'http://example.com/foo-bar')->getContent();
+
+        $delay = $logger->context['delay'] ?? null;
+
+        $this->assertArrayHasKey('delay', $logger->context);
+        $this->assertNotNull($delay);
+        $this->assertSame((int) ($retryAfter * 1000), $delay);
+    }
 }