Skip to content

Commit 770f0c0

Browse files
committed
Require Undertow 1.3.5+, Tyrus 1.11+, Jetty 9.3+, Tomcat 8.5+
Issue: SPR-13495
1 parent ccf791b commit 770f0c0

File tree

11 files changed

+126
-823
lines changed

11 files changed

+126
-823
lines changed

spring-instrument-tomcat/src/main/java/org/apache/catalina/loader/WebappClassLoader.java

Lines changed: 0 additions & 44 deletions
This file was deleted.

spring-instrument-tomcat/src/main/java/org/springframework/instrument/classloading/WeavingTransformer.java

Lines changed: 0 additions & 107 deletions
This file was deleted.

spring-instrument-tomcat/src/main/java/org/springframework/instrument/classloading/tomcat/TomcatInstrumentableClassLoader.java

Lines changed: 0 additions & 166 deletions
This file was deleted.

spring-instrument-tomcat/src/main/java/overview.html

Lines changed: 0 additions & 7 deletions
This file was deleted.

spring-instrument-tomcat/src/test/java/.gitignore

Whitespace-only changes.

spring-instrument-tomcat/src/test/resources/log4j.properties

Lines changed: 0 additions & 10 deletions
This file was deleted.

spring-websocket/src/main/java/org/springframework/web/socket/server/jetty/JettyRequestUpgradeStrategy.java

Lines changed: 4 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2002-2015 the original author or authors.
2+
* Copyright 2002-2016 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.
@@ -56,19 +56,15 @@
5656
import org.springframework.web.socket.server.RequestUpgradeStrategy;
5757

5858
/**
59-
* A {@link RequestUpgradeStrategy} for use with Jetty 9.0-9.3. Based on Jetty's
60-
* internal {@code org.eclipse.jetty.websocket.server.WebSocketHandler} class.
59+
* A {@link RequestUpgradeStrategy} for use with Jetty 9.3 and higher. Based on
60+
* Jetty's internal {@code org.eclipse.jetty.websocket.server.WebSocketHandler} class.
6161
*
6262
* @author Phillip Webb
6363
* @author Rossen Stoyanchev
6464
* @since 4.0
6565
*/
6666
public class JettyRequestUpgradeStrategy implements RequestUpgradeStrategy, Lifecycle, ServletContextAware {
6767

68-
// Pre-Jetty 9.3 init method without ServletContext
69-
private static final Method webSocketFactoryInitMethod =
70-
ClassUtils.getMethodIfAvailable(WebSocketServerFactory.class, "init");
71-
7268
private static final ThreadLocal<WebSocketHandlerContainer> wsContainerHolder =
7369
new NamedThreadLocal<WebSocketHandlerContainer>("WebSocket Handler Container");
7470

@@ -153,12 +149,7 @@ public void start() {
153149
if (!isRunning()) {
154150
this.running = true;
155151
try {
156-
if (webSocketFactoryInitMethod != null) {
157-
webSocketFactoryInitMethod.invoke(this.factory);
158-
}
159-
else {
160-
this.factory.init(this.servletContext);
161-
}
152+
this.factory.init(this.servletContext);
162153
}
163154
catch (Exception ex) {
164155
throw new IllegalStateException("Unable to initialize Jetty WebSocketServerFactory", ex);

0 commit comments

Comments
 (0)