feat: retry on 408 Request Timeout (#125)

lithic-com · Sep 15, 2023 · c9258c8 · c9258c8
1 parent dfac218
commit c9258c8
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/README.md b/README.md
@@ -241,8 +241,8 @@ client.Cards.List(
 ## Retries
 
 Certain errors will be automatically retried 2 times by default, with a short exponential backoff.
-Connection errors (for example, due to a network connectivity problem), 409 Conflict, 429 Rate Limit,
-and >=500 Internal errors will all be retried by default.
+Connection errors (for example, due to a network connectivity problem), 408 Request Timeout, 409 Conflict,
+429 Rate Limit, and >=500 Internal errors will all be retried by default.
 
 You can use the `WithMaxRetries` option to configure or disable this:
 

diff --git a/internal/requestconfig/requestconfig.go b/internal/requestconfig/requestconfig.go
@@ -192,6 +192,7 @@ func (cfg *RequestConfig) Execute() error {
 		}
 
 		shouldRetry := err != nil ||
+			res.StatusCode == http.StatusRequestTimeout ||
 			res.StatusCode == http.StatusConflict ||
 			res.StatusCode == http.StatusTooManyRequests ||
 			res.StatusCode >= http.StatusInternalServerError