Skip to content

Release closing connections from client pool - #1042

Merged
hamnis merged 1 commit into
http4s:series/0.7from
cwe-dixa:bugfix/release-closing-connections
Oct 27, 2026
Merged

hamnis merged 1 commit into
http4s:series/0.7from
cwe-dixa:bugfix/release-closing-connections

Conversation

@cwe-dixa

Copy link
Copy Markdown
Contributor

If a request returns Connection: close, we should not keep the connection in the client pool.

Implementation ended up going through a netty attribute since the netty isKeepAlive utility method requires a netty http response object.

If a request returns `Connection: close`, we should not keep the
connection in the client pool.

Implementation ended up going through a netty attribute since the netty
isKeepAlive utility method requires a netty http response object.
@cwe-dixa
cwe-dixa marked this pull request as ready for review October 27, 2026 10:55
@cwe-dixa

Copy link
Copy Markdown
Contributor Author

Any chance we can do a patch release for this one? 🙏

@hamnis
hamnis merged commit e49df5b into http4s:series/0.7 Oct 27, 2026
7 checks passed
@cwe-dixa
cwe-dixa deleted the bugfix/release-closing-connections branch October 27, 2026 12:00
@hamnis

hamnis commented Oct 27, 2026

Copy link
Copy Markdown
Collaborator

Publishing 0.7.4 to maven central now

@cwe-dixa

Copy link
Copy Markdown
Contributor Author

Thank you 🙏

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.

2 participants