Skip to content

Honor configured :retry_delay over Retry-After#535

Open
Sim-hu wants to merge 1 commit into
wojtekmach:mainfrom
Sim-hu:fix-retry-delay-precedence
Open

Honor configured :retry_delay over Retry-After#535
Sim-hu wants to merge 1 commit into
wojtekmach:mainfrom
Sim-hu:fix-retry-delay-precedence

Conversation

@Sim-hu

@Sim-hu Sim-hu commented Mar 27, 2026

Copy link
Copy Markdown

Fixes #463.

The docs say :retry_delay should take precedence when it is explicitly set,
but 429/503 responses were still using Retry-After first.

This changes the retry delay lookup so that an explicit :retry_delay wins,
and Retry-After is only used when :retry_delay is not set.

Also adds a regression test for the case where both are present.

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.

:retry_after ignored if there is a Retry-After response header

1 participant