Skip to content

Fix POST retries and 504 retry classification#624

Merged
crmne merged 1 commit intomainfrom
integrate-pr523
Feb 20, 2026
Merged

Fix POST retries and 504 retry classification#624
crmne merged 1 commit intomainfrom
integrate-pr523

Conversation

@crmne
Copy link
Owner

@crmne crmne commented Feb 20, 2026

This follows up on #523 with a minimal patch set:

  • enable retries for POST while preserving Faraday default idempotent methods
  • map HTTP 504 to RubyLLM::ServiceUnavailableError (so retry classification is consistent)
  • reduce retry backoff settings in test config to avoid long sleeps in retry/error specs
  • add focused specs for retry config and 502/503/504 error mapping

Related: #417

@crmne crmne merged commit 984580d into main Feb 20, 2026
2 of 21 checks passed
@crmne crmne deleted the integrate-pr523 branch February 20, 2026 09:12
@crmne crmne linked an issue Feb 20, 2026 that may be closed by this pull request
3 tasks
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.

[BUG] RubyLLM not retrying POST requests

1 participant