Skip to content

Rewind request bodies before retry#229

Merged
ioquatix merged 3 commits into
mainfrom
retry-rewinds-request-body
Jul 24, 2026
Merged

Rewind request bodies before retry#229
ioquatix merged 3 commits into
mainfrom
retry-rewinds-request-body

Conversation

@ioquatix

@ioquatix ioquatix commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

  • Use request.rewind! before retrying requests known not to have been processed (Protocol::HTTP::RefusedError).
  • Use request.retry! before retrying ambiguous connection failures.
  • Require released protocol-http ~> 0.64 for the retry preparation API.
  • Add focused client retry tests proving consumed request bodies are resent on retry.

Testing

  • bundle exec sus test/async/http/client.rb
  • bundle exec sus

Full suite result: 231 passed, 3 skipped.

@ioquatix
ioquatix merged commit 0b2a7ca into main Jul 24, 2026
34 of 40 checks passed
@ioquatix
ioquatix deleted the retry-rewinds-request-body branch July 24, 2026 21:42
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.

1 participant