Skip to content

Reset HTTP client connection after unknown error responses#201

Merged
aonemd merged 4 commits intomasterfrom
fix/reset-http-client-on-unknown-response
Apr 2, 2026
Merged

Reset HTTP client connection after unknown error responses#201
aonemd merged 4 commits intomasterfrom
fix/reset-http-client-on-unknown-response

Conversation

@Keith-wright
Copy link
Copy Markdown
Contributor

@Keith-wright Keith-wright commented Apr 1, 2026

Summary

  • Calls reset_all on the HTTP client after receiving an unknown/unexpected status code, ensuring subsequent requests get a clean connection
  • Known response codes handled by the library (200, 202, 400, 401, 403, 404, 407, 413) are treated as clean HTTP exchanges and do not trigger a reset
  • Fixes an issue where an unexpected server response could leave the connection in a state that caused the next request to fail

Thanks to @rmosolgo for reporting and following up on this.

Closes #193

Changelog

  • [FIXED] Reset HTTP client connection after unknown error responses to prevent subsequent requests from failing.

Copy link
Copy Markdown
Member

@aonemd aonemd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @Keith-wright!

@aonemd aonemd added the release-minor Minor version bump label Apr 2, 2026
@aonemd aonemd merged commit 7b24c6e into master Apr 2, 2026
3 checks passed
@aonemd aonemd deleted the fix/reset-http-client-on-unknown-response branch April 2, 2026 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release-minor Minor version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Requesting channel_info without info: raises an error from the previous request

4 participants