Skip to content

feat: exponential backoff on consecutive 429s from Parcel API - #86

Merged
jmdevita merged 1 commit into
mainfrom
feat/exponential-backoff-429
May 16, 2026
Merged

jmdevita merged 1 commit into
mainfrom
feat/exponential-backoff-429

Conversation

@jmdevita

Copy link
Copy Markdown
Owner

Doubles the retry interval on each consecutive 429 (300s -> 600s -> 1200s ...), floored at the user's configured interval and capped at 1h. Resets to zero on the first successful response. Prevents the integration from continuing to poll at the same cadence when the Parcel API is enforcing a rolling/burst rate limit beyond the documented 20/hr.

Doubles the retry interval on each consecutive 429 (300s -> 600s ->
1200s ...), floored at the user's configured interval and capped at
1h. Resets to zero on the first successful response. Prevents the
integration from continuing to poll at the same cadence when the
Parcel API is enforcing a rolling/burst rate limit beyond the
documented 20/hr.
@jmdevita

jmdevita commented May 16, 2026

Copy link
Copy Markdown
Owner Author

Hey @PineappleEmperor ! If you have a moment to check this PR out, if not, no worries. Hope you're doing well! :)

@PineappleEmperor

PineappleEmperor commented May 16, 2026

Copy link
Copy Markdown
Collaborator

Hallo! I'm alive! Hope all is well with you! A super-quick-10s review is: I can't quite see how it resets the update interval after a successful response? looks like I missed line 157 where it resets to 0! 🙃 LGTM!

@PineappleEmperor PineappleEmperor left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@jmdevita
jmdevita merged commit 316f8fc into main May 16, 2026
8 checks passed
@jmdevita
jmdevita deleted the feat/exponential-backoff-429 branch May 16, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants