fix java 11

Author: arturobernalg

httpclient5/src/main/java/org/apache/hc/client5/http/config/EnvironmentProxyConfigurer.java

@@ -75,7 +75,10 @@ public static void apply() {
         if (noProxy != null && System.getProperty("http.nonProxyHosts") == null) {
             final String list = noProxy.replace(',', '|');
             setProperty("http.nonProxyHosts", list);
-            setProperty("https.nonProxyHosts", list);
+            /* only write HTTPS when it is still unset  */
+            if (System.getProperty("https.nonProxyHosts") == null) {   // ← FIX
+                setProperty("https.nonProxyHosts", list);
+            }
         }
     }
 

httpclient5/src/test/java/org/apache/hc/client5/http/config/EnvironmentProxyConfigurerTest.java

@@ -99,24 +99,20 @@ void does_not_overwrite_already_set_properties() throws Exception {
     void translates_no_proxy_to_pipe_delimited_hosts() throws Exception {
         backup("http.nonProxyHosts", "https.nonProxyHosts");
 
+        // ensure both props are null before we invoke the bridge
+        System.clearProperty("http.nonProxyHosts");
+        System.clearProperty("https.nonProxyHosts");
+
         withEnvironmentVariable("NO_PROXY", "localhost,127.0.0.1")
                 .execute(() -> {
-                    final String beforeHttp = System.getProperty("http.nonProxyHosts");
-                    final String beforeHttps = System.getProperty("https.nonProxyHosts");
-
                     EnvironmentProxyConfigurer.apply();
-
-                    final String expected = "localhost|127.0.0.1";
-                    assertEquals(
-                            beforeHttp != null ? beforeHttp : expected,
+                    assertEquals("localhost|127.0.0.1",
                             System.getProperty("http.nonProxyHosts"));
-                    assertEquals(
-                            beforeHttps != null ? beforeHttps : expected,
+                    assertEquals("localhost|127.0.0.1",
                             System.getProperty("https.nonProxyHosts"));
                 });
     }
 
-
     @Test
     void noop_when_no_relevant_env_vars() {
         backup("http.proxyHost");