Merge pull request #3 from wiz-sec/danny/WZ-42693-use-retry-after-only-if-lower-than-max

dany74q · web-flow · commit aa69c2701c46 · 2025-03-19T15:05:48.000+02:00
danny/WZ 42693 use retry after only if lower than max
diff --git a/client.go b/client.go
@@ -552,7 +552,9 @@ func DefaultBackoff(min, max time.Duration, attemptNum int, resp *http.Response)
 	if resp != nil {
 		if resp.StatusCode == http.StatusTooManyRequests || resp.StatusCode == http.StatusServiceUnavailable {
 			if sleep, ok := parseRetryAfterHeader(resp.Header["Retry-After"]); ok {
-				return sleep
+				if sleep <= max {
+					return sleep
+				}
 			}
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -552,7 +552,9 @@ func DefaultBackoff(min, max time.Duration, attemptNum int, resp *http.Response)`
`552`	`552`	`if resp != nil {`
`553`	`553`	`if resp.StatusCode == http.StatusTooManyRequests \|\| resp.StatusCode == http.StatusServiceUnavailable {`
`554`	`554`	`if sleep, ok := parseRetryAfterHeader(resp.Header["Retry-After"]); ok {`
`555`		`- return sleep`
	`555`	`+ if sleep <= max {`
	`556`	`+ return sleep`
	`557`	`+ }`
`556`	`558`	`}`
`557`	`559`	`}`
`558`	`560`	`}`