Merge pull request #184 from andyexeter/dynamic-retry-after

tareqtms · web-flow · commit cdcb8dc69990 · 2020-12-05T14:29:56.000+06:00
Use Retry-After header as value to sleep for before retrying request
diff --git a/lib/CurlRequest.php b/lib/CurlRequest.php
@@ -166,7 +166,13 @@ protected static function processRequest($ch)
                 throw new ResourceRateLimitException($response->getBody());
             }
 
-            usleep(500000);
+            $retryAfter = $response->getHeader('Retry-After');
+
+            if ($retryAfter === null) {
+                break;
+            }
+
+            sleep((float)$retryAfter);
         }
 
         if (curl_errno($ch)) {

Original file line number	Diff line number	Diff line change
`@@ -166,7 +166,13 @@ protected static function processRequest($ch)`
`166`	`166`	`throw new ResourceRateLimitException($response->getBody());`
`167`	`167`	`}`
`168`	`168`
`169`		`- usleep(500000);`
	`169`	`+ $retryAfter = $response->getHeader('Retry-After');`
	`170`	`+`
	`171`	`+ if ($retryAfter === null) {`
	`172`	`+ break;`
	`173`	`+ }`
	`174`	`+`
	`175`	`+ sleep((float)$retryAfter);`
`170`	`176`	`}`
`171`	`177`
`172`	`178`	`if (curl_errno($ch)) {`