Use new JettyClientHttpConnector constructor

ielatif · philwebb · commit 4ac1407af15f · 2019-05-31T15:31:41.000-07:00
Simplify `ClientHttpConnectorConfiguration`by using the recently introduced `JettyClientHttpConnector` constructor. See gh-17015
diff --git a/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/function/client/ClientHttpConnectorConfiguration.java b/spring-boot-project/spring-boot-autoconfigure/src/main/java/org/springframework/boot/autoconfigure/web/reactive/function/client/ClientHttpConnectorConfiguration.java
@@ -78,10 +78,7 @@ public JettyClientHttpConnector jettyClientHttpConnector(
 				JettyResourceFactory jettyResourceFactory) {
 			SslContextFactory sslContextFactory = new SslContextFactory.Client();
 			HttpClient httpClient = new HttpClient(sslContextFactory);
-			httpClient.setExecutor(jettyResourceFactory.getExecutor());
-			httpClient.setByteBufferPool(jettyResourceFactory.getByteBufferPool());
-			httpClient.setScheduler(jettyResourceFactory.getScheduler());
-			return new JettyClientHttpConnector(httpClient);
+			return new JettyClientHttpConnector(httpClient, jettyResourceFactory);
 		}
 
 	}

Original file line number	Diff line number	Diff line change
`@@ -78,10 +78,7 @@ public JettyClientHttpConnector jettyClientHttpConnector(`
`78`	`78`	`JettyResourceFactory jettyResourceFactory) {`
`79`	`79`	`SslContextFactory sslContextFactory = new SslContextFactory.Client();`
`80`	`80`	`HttpClient httpClient = new HttpClient(sslContextFactory);`
`81`		`- httpClient.setExecutor(jettyResourceFactory.getExecutor());`
`82`		`- httpClient.setByteBufferPool(jettyResourceFactory.getByteBufferPool());`
`83`		`- httpClient.setScheduler(jettyResourceFactory.getScheduler());`
`84`		`- return new JettyClientHttpConnector(httpClient);`
	`81`	`+ return new JettyClientHttpConnector(httpClient, jettyResourceFactory);`
`85`	`82`	`}`
`86`	`83`
`87`	`84`	`}`