jgoodyear commented on code in PR #2069:
URL: https://github.com/apache/cxf/pull/2069#discussion_r1764668705
##########
rt/transports/http/src/main/java/org/apache/cxf/transport/http/HttpClientHTTPConduit.java:
##########
@@ -243,13 +243,18 @@ private boolean isSslTargetDifferent(URI lastURL, URI
url) {
|| !lastURL.getHost().equals(url.getHost())
|| lastURL.getPort() != url.getPort();
}
-
+
@Override
public void close(Message msg) throws IOException {
+ OutputStream os = msg.getContent(OutputStream.class);
+ // Java 21 may hang on close, we flush stream to help close them out.
+ if (os != null &&
AutoCloseable.class.isAssignableFrom(HttpClient.class)) {
Review Comment:
I updated the branch last night with an impl that retains the flush but
doesn't harm the NettyHttpConduit
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
