Skip to content

Prevent negative Retry-After headers - #427

Merged
robgruen merged 2 commits into
mainfrom
dev/robgruen/retry_after_fix
Aug 21, 2026
Merged

Prevent negative Retry-After headers#427
robgruen merged 2 commits into
mainfrom
dev/robgruen/retry_after_fix

Conversation

@robgruen

Copy link
Copy Markdown
Contributor

This pull request introduces enhancements to the way retry delays are handled and tested in the codebase, specifically focusing on simulating and verifying retry logic for API calls. The main changes include improvements to the test suite for retry behavior and better simulation of asynchronous delays.

Testing improvements:

  • Added a new test to simulate and verify the retry-after delay mechanism in the API call logic, including mocking setTimeout to track scheduled delays and ensure correct behavior after retries.

Retry logic enhancements:

  • Improved handling of retry delays by capturing and asserting the correct delay values when a retry is triggered, ensuring that the logic for retrying after a failure is robust and testable.

@robgruen
robgruen merged commit d45028c into mainAug 21, 2026
13 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@robgruen@TalZaccai