Fix HttpClientConfig copying issue in MonoHttpConnect (#3964)

Ensure all changes are made to the copy instead of the original configuration

Signed-off-by: Violeta Georgieva <[email protected]>

Author: violetagg

reactor-netty-http/src/main/java/reactor/netty/http/client/HttpClientConnect.java

@@ -220,12 +220,16 @@ public void subscribe(CoreSubscriber<? super Connection> actual) {
 				if (handler.toURI.isSecure()) {
 					if (_config.sslProvider == null) {
 						configCopied = true;
-						_config = new HttpClientConfig(config);
+						_config = new HttpClientConfig(_config);
 						_config.sslProvider = HttpClientSecure.defaultSslProvider(_config);
 					}
 
 					if (_config.checkProtocol(HttpClientConfig.h2c)) {
 						if (_config.protocols.length > 1) {
+							if (!configCopied) {
+								configCopied = true;
+								_config = new HttpClientConfig(_config);
+							}
 							removeIncompatibleProtocol(_config, HttpProtocol.H2C);
 						}
 						else {
@@ -240,12 +244,16 @@ public void subscribe(CoreSubscriber<? super Connection> actual) {
 				else {
 					if (_config.sslProvider != null) {
 						configCopied = true;
-						_config = new HttpClientConfig(config);
+						_config = new HttpClientConfig(_config);
 						_config.sslProvider = null;
 					}
 
 					if (_config.checkProtocol(HttpClientConfig.h2)) {
 						if (_config.protocols.length > 1) {
+							if (!configCopied) {
+								configCopied = true;
+								_config = new HttpClientConfig(_config);
+							}
 							removeIncompatibleProtocol(_config, HttpProtocol.H2);
 						}
 						else {
@@ -266,7 +274,7 @@ else if (_config.checkProtocol(HttpClientConfig.h3)) {
 					if (proxyProviderSupplier != null) {
 						if (!configCopied) {
 							configCopied = true;
-							_config = new HttpClientConfig(config);
+							_config = new HttpClientConfig(_config);
 						}
 						ProxyProvider proxyProvider = proxyProviderSupplier.get();
 						_config.proxyProvider(proxyProvider);