Skip to content

Map remote connection failures - #58

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
samuel/remote-connection-errors
Aug 19, 2026
Merged

Map remote connection failures#58
samuel-williams-shopify merged 1 commit into
mainfrom
samuel/remote-connection-errors

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • Map Errno::EPIPE and Errno::ECONNRESET from the HTTP/1 connection stream to Protocol::HTTP::RemoteError.
  • Preserve the original socket exception as the cause.
  • Mark affected connections non-persistent so they cannot be reused.
  • Continue treating a reset while waiting for a request line as EOF, while invalidating the connection.

Testing

  • bundle exec bake test
  • bundle exec rubocop
  • bundle exec bake decode:index:coverage lib

@samuel-williams-shopify
samuel-williams-shopify force-pushed the samuel/remote-connection-errors branch from 178321d to 9a17e1f Compare August 19, 2026 23:01
@samuel-williams-shopify
samuel-williams-shopify merged commit 1218eed into main Aug 19, 2026
36 of 40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the samuel/remote-connection-errors branch August 19, 2026 23:04
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