Skip to content

Commit 51092af

Browse files
committed
Polish
1 parent 0b9667c commit 51092af

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

spring-boot-autoconfigure/src/test/java/org/springframework/boot/autoconfigure/jersey/JerseyAutoConfigurationServletContainerTests.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121

2222
import org.apache.catalina.Context;
2323
import org.apache.catalina.Wrapper;
24+
import org.apache.tomcat.util.buf.UDecoder;
2425
import org.glassfish.jersey.server.ResourceConfig;
2526
import org.glassfish.jersey.servlet.ServletContainer;
2627
import org.junit.ClassRule;
@@ -104,7 +105,8 @@ protected void postProcessContext(Context context) {
104105
jerseyServlet.setServlet(new ServletContainer());
105106
jerseyServlet.setOverridable(false);
106107
context.addChild(jerseyServlet);
107-
context.addServletMapping("/*", servletName);
108+
String pattern = UDecoder.URLDecode("/*", "UTF-8");
109+
context.addServletMappingDecoded(pattern, servletName);
108110
}
109111

110112
};

spring-boot/src/main/java/org/springframework/boot/context/embedded/tomcat/TomcatEmbeddedServletContainerFactory.java

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -254,7 +254,7 @@ private void addDefaultServlet(Context context) {
254254
// Otherwise the default location of a Spring DispatcherServlet cannot be set
255255
defaultServlet.setOverridable(true);
256256
context.addChild(defaultServlet);
257-
context.addServletMapping("/", "default");
257+
addServletMapping(context, "/", "default");
258258
}
259259

260260
private void addJspServlet(Context context) {
@@ -268,8 +268,13 @@ private void addJspServlet(Context context) {
268268
}
269269
jspServlet.setLoadOnStartup(3);
270270
context.addChild(jspServlet);
271-
context.addServletMapping("*.jsp", "jsp");
272-
context.addServletMapping("*.jspx", "jsp");
271+
addServletMapping(context, "*.jsp", "jsp");
272+
addServletMapping(context, "*.jspx", "jsp");
273+
}
274+
275+
@SuppressWarnings("deprecation")
276+
private void addServletMapping(Context context, String pattern, String name) {
277+
context.addServletMapping(pattern, name);
273278
}
274279

275280
private void addJasperInitializer(TomcatEmbeddedContext context) {

0 commit comments

Comments
 (0)