ffang opened a new pull request, #3154:
URL: https://github.com/apache/cxf/pull/3154
…tem proxy properties
## Summary
`ProxyFactory` was reading `http.proxyHost`, `http.proxyPort`, and
`http.nonProxyHosts`
directly from system properties at construction time, bypassing Java's
`ProxySelector`
mechanism entirely. This prevented applications from using custom
`ProxySelector`
implementations — for example, wrapping `sun.net.spi.DefaultProxySelector`
to selectively
bypass the proxy for specific hosts.
The fix removes the system-property reading from `ProxyFactory` and
delegates to
`ProxySelector.getDefault()` instead. When no custom `ProxySelector` is
installed the
behaviour is identical to before (`DefaultProxySelector` reads the same
system properties
internally). When a custom `ProxySelector` is installed it is now consulted
as expected.
The explicit `HTTPClientPolicy.setProxyServer(...)` path is unaffected.
## Behaviour notes
- **Default behaviour unchanged** — `DefaultProxySelector` reads
`http.proxyHost` /
`http.proxyPort` / `http.nonProxyHosts` exactly as before.
- Both `HttpClientHTTPConduit` and `URLConnectionHTTPConduit` benefit from
this fix.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
