Skip to content

fix: clamp negative Retry-After delays before sleeping - #855

Open
abhinavkr26104 wants to merge 1 commit into
openai:mainfrom
abhinavkr26104:codex/fix-negative-retry-after
Open

fix: clamp negative Retry-After delays before sleeping#855
abhinavkr26104 wants to merge 1 commit into
openai:mainfrom
abhinavkr26104:codex/fix-negative-retry-after

Conversation

@abhinavkr26104

Copy link
Copy Markdown

Summary

  • clamp parsed Retry-After and Retry-After-Ms delays to Duration.ZERO when they are negative
  • preserve immediate retries for past HTTP-date values
  • add sync and async regression coverage for negative numeric delays and past dates

Fixes #853

Testing

  • git diff --check passed
  • direct ktfmt 0.61 validation passed for both changed Kotlin files
  • full Gradle test suite was attempted online and offline; the online run timed out during Kotlin/Gradle execution without diagnostics, while the offline run was blocked by uncached junit-jupiter-engine:5.9.3
  • repository lint tasks were attempted; direct ktfmt passed, while Gradle lint was blocked by the same local dependency/compile environment

@abhinavkr26104
abhinavkr26104 requested a review from a team as a code owner August 10, 2026 07:16
Sign up for free to 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.

fix: clamp negative Retry-After delays before sleeping

1 participant