fix: parsing millisecond precision time for sleeping when rate limited

VetSoftKeith · VetSoftKeith · commit 3d4bf0e1dc21 · 2022-03-05T22:25:07.000-05:00
diff --git a/api/rate_limits.go b/api/rate_limits.go
@@ -27,9 +27,9 @@ import (
 )
 
 type rateLimitResponse struct {
-	Message    string        `json:"message"`
-	RetryAfter time.Duration `json:"retry_after"`
-	Global     bool          `json:"global"`
+	Message    string  `json:"message"`
+	RetryAfter float64 `json:"retry_after"`
+	Global     bool    `json:"global"`
 }
 
 // customRateLimit holds information for defining a custom rate limit
diff --git a/api/rest.go b/api/rest.go
@@ -137,7 +137,7 @@ func (r *RateLimiter) lockedRequest(method, route, contentType string, b interfa
 			return nil, err
 		}
 
-		time.Sleep(rlr.RetryAfter)
+		time.Sleep(time.Duration(rlr.RetryAfter * float64(time.Second)))
 
 		return r.lockedRequest(method, route, contentType, b, r.lockBucketObject(bucket), sequence, reason)
 	}

Original file line number	Diff line number	Diff line change
`@@ -27,9 +27,9 @@ import (`
`27`	`27`	`)`
`28`	`28`
`29`	`29`	`type rateLimitResponse struct {`
`30`		- Message string `json:"message"`
`31`		- RetryAfter time.Duration `json:"retry_after"`
`32`		- Global bool `json:"global"`
	`30`	+ Message string `json:"message"`
	`31`	+ RetryAfter float64 `json:"retry_after"`
	`32`	+ Global bool `json:"global"`
`33`	`33`	`}`
`34`	`34`
`35`	`35`	`// customRateLimit holds information for defining a custom rate limit`
Original file line number	Diff line number	Diff line change
`@@ -137,7 +137,7 @@ func (r *RateLimiter) lockedRequest(method, route, contentType string, b interfa`
`137`	`137`	`return nil, err`
`138`	`138`	`}`
`139`	`139`
`140`		`- time.Sleep(rlr.RetryAfter)`
	`140`	`+ time.Sleep(time.Duration(rlr.RetryAfter * float64(time.Second)))`
`141`	`141`
`142`	`142`	`return r.lockedRequest(method, route, contentType, b, r.lockBucketObject(bucket), sequence, reason)`
`143`	`143`	`}`