Fixes max_duration not being respected

As reported on #46316, when a request gets a successful response in further attempts, the max_duration is not respected and the request may take longer than the value specified here. My way to solve the issue was by reducing the timeout option in future requests, making it smaller in every attempt. By doing so, the total time of a $client->request will never be much longer than max_duration. Perhaps a second or two maybe, due to the waiting time between requests, but not as much as the timeout.
symfony · r-martins · May 12, 2022 · May 12, 2022 · deb67aa945061bee838de68de37c69ac7fb3086e
commit deb67aa945061bee838de68de37c69ac7fb3086e
diff --git a/src/Symfony/Component/HttpClient/Response/AsyncContext.php b/src/Symfony/Component/HttpClient/Response/AsyncContext.php
@@ -159,6 +159,14 @@ public function replaceRequest(string $method, string $url, array $options = [])
                $onProgress($dlNow, $dlSize, $thisInfo + $info);
            };
        }
+
+        if ($maxDuration = $options['max_duration'] ?? false) {
+            $totalElapsed = 0;
+            foreach ($this->info['previous_info'] as $req) {
+                $totalElapsed += $req['total_time'];
+            }
+            $options['timeout'] = ceil($maxDuration - $totalElapsed);
+        }

        return $this->response = $this->client->request($method, $url, ['buffer' => false] + $options);
    }