reta commented on code in PR #1380:
URL: https://github.com/apache/cxf/pull/1380#discussion_r1292355008
##########
rt/transports/http/src/main/java/org/apache/cxf/transport/http/HttpClientHTTPConduit.java:
##########
@@ -303,19 +303,34 @@ protected OutputStream createOutputStream(Message
message, boolean needToCacheRe
* This class <i>must</i> be static so it doesn't capture a reference to
{@code HttpClientHTTPConduit.this} and
* through that to {@link HttpClientHTTPConduit#client}. Otherwise the
client can never be garbage collected, which
* means that the companion "SelectorManager" thread keeps running
indefinitely (see CXF-8885).
+ * <p>
+ * The {@link HTTPClientPolicy} must be copied here because its property
change listeners can still have hard
+ * references to the {@code HttpClientHTTPConduit}.
*/
private static final class ProxyFactoryProxySelector extends ProxySelector
{
private final ProxyFactory proxyFactory;
- private final HTTPClientPolicy csPolicy;
+ private final HTTPClientPolicy clientPolicy;
ProxyFactoryProxySelector(ProxyFactory proxyFactory, HTTPClientPolicy
csPolicy) {
this.proxyFactory = proxyFactory;
- this.csPolicy = csPolicy;
+ this.clientPolicy = new HTTPClientPolicy();
Review Comment:
Thank you @LeoWoerteler , I think it is sufficient to see the problem, will
try to look shortly
--
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
