[resteasy-commits] [resteasy/Resteasy] 63c091: [RESTEASY-2486] Fix for RestEasyClientBuilder http...

[Christos Sotiriou]

  Branch: refs/heads/master
  Home:   https://github.com/resteasy/Resteasy
  Commit: 63c091c6da88b69ad9508bfa709a539ab77f25c7
      https://github.com/resteasy/Resteasy/commit/63c091c6da88b69ad9508bfa709a539ab77f25c7
  Author: Christos Sotiriou <csotiriou at users.noreply.github.com>
  Date:   2020-01-29 (Wed, 29 Jan 2020)

  Changed paths:
    M resteasy-client-microprofile/src/main/java/org/jboss/resteasy/microprofile/client/RestClientBuilderImpl.java
    M resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/engines/URLConnectionClientEngineBuilder.java
    M resteasy-client/src/main/java/org/jboss/resteasy/client/jaxrs/engines/URLConnectionEngine.java

  Log Message:
  -----------
  [RESTEASY-2486] Fix for RestEasyClientBuilder http proxy not working (#2268)

* check resetProxy flag when deciding whether to initialize a new http engine


Only setup a new http engine if a new proxy has been set up.

Avoids setting up a new client each time that the proxy is null (which was introduced with the previous commit)

* Revert "check resetProxy flag when deciding whether to initialize a new http engine"

This reverts commit dd8c8cbf648d021076a4ee51feee52efcdc3da68.

* Added support for proxy in URLConnectionEngine

* take into account the resteasyClientBuilder's default proxy settings when building

* when building the microprofile client instance, take into account user-defined proxy properties, and pass them to the resteasyClientBuilder instance

* Only check for Rest client Builder properties if no env property is set.

* fixed forgotten setter for proxy in the case where a proxy is set