Skip to content

Commit 9e2e824

Browse files
committed
Upgrade to Tomcat 8.5.11
Closes gh-7360
1 parent 0ce7c95 commit 9e2e824

File tree

2 files changed

+11
-9
lines changed

2 files changed

+11
-9
lines changed

spring-boot-dependencies/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@
176176
<thymeleaf-layout-dialect.version>1.4.0</thymeleaf-layout-dialect.version>
177177
<thymeleaf-extras-data-attribute.version>1.3</thymeleaf-extras-data-attribute.version>
178178
<thymeleaf-extras-java8time.version>2.1.0.RELEASE</thymeleaf-extras-java8time.version>
179-
<tomcat.version>8.5.6</tomcat.version>
179+
<tomcat.version>8.5.11</tomcat.version>
180180
<undertow.version>1.3.25.Final</undertow.version>
181181
<velocity.version>1.7</velocity.version>
182182
<velocity-tools.version>2.0</velocity-tools.version>

spring-boot/src/test/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactoryTests.java

Lines changed: 10 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2012-2016 the original author or authors.
2+
* Copyright 2012-2017 the original author or authors.
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -246,14 +246,18 @@ public void uriEncoding() throws Exception {
246246
TomcatEmbeddedServletContainerFactory factory = getFactory();
247247
factory.setUriEncoding(Charset.forName("US-ASCII"));
248248
Tomcat tomcat = getTomcat(factory);
249-
assertThat(tomcat.getConnector().getURIEncoding()).isEqualTo("US-ASCII");
249+
Connector connector = ((TomcatEmbeddedServletContainer) this.container)
250+
.getServiceConnectors().get(tomcat.getService())[0];
251+
assertThat(connector.getURIEncoding()).isEqualTo("US-ASCII");
250252
}
251253

252254
@Test
253255
public void defaultUriEncoding() throws Exception {
254256
TomcatEmbeddedServletContainerFactory factory = getFactory();
255257
Tomcat tomcat = getTomcat(factory);
256-
assertThat(tomcat.getConnector().getURIEncoding()).isEqualTo("UTF-8");
258+
Connector connector = ((TomcatEmbeddedServletContainer) this.container)
259+
.getServiceConnectors().get(tomcat.getService())[0];
260+
assertThat(connector.getURIEncoding()).isEqualTo("UTF-8");
257261
}
258262

259263
@Test
@@ -262,13 +266,12 @@ public void sslCiphersConfiguration() throws Exception {
262266
ssl.setKeyStore("test.jks");
263267
ssl.setKeyStorePassword("secret");
264268
ssl.setCiphers(new String[] { "ALPHA", "BRAVO", "CHARLIE" });
265-
266269
TomcatEmbeddedServletContainerFactory factory = getFactory();
267270
factory.setSsl(ssl);
268271

269272
Tomcat tomcat = getTomcat(factory);
270-
Connector connector = tomcat.getConnector();
271-
273+
Connector connector = ((TomcatEmbeddedServletContainer) this.container)
274+
.getServiceConnectors().get(tomcat.getService())[0];
272275
SSLHostConfig[] sslHostConfigs = connector.getProtocolHandler()
273276
.findSslHostConfigs();
274277
assertThat(sslHostConfigs[0].getCiphers()).isEqualTo("ALPHA:BRAVO:CHARLIE");
@@ -308,9 +311,8 @@ public void sslEnabledProtocolsConfiguration() throws Exception {
308311
this.container = factory
309312
.getEmbeddedServletContainer(sessionServletRegistration());
310313
Tomcat tomcat = ((TomcatEmbeddedServletContainer) this.container).getTomcat();
311-
Connector connector = tomcat.getConnector();
312-
313314
this.container.start();
315+
Connector connector = tomcat.getConnector();
314316
SSLHostConfig sslHostConfig = connector.getProtocolHandler()
315317
.findSslHostConfigs()[0];
316318
assertThat(sslHostConfig.getSslProtocol()).isEqualTo("TLS");

0 commit comments

Comments
 (0)