mirror of https://github.com/apache/activemq.git
Fix some itests due to the servlet import range being inadequate for the version of karaf we are testing with
This commit is contained in:
parent
d4aff4da64
commit
2128d4d246
|
@ -21,6 +21,8 @@ import java.net.InetSocketAddress;
|
||||||
import java.net.URI;
|
import java.net.URI;
|
||||||
import java.util.Map;
|
import java.util.Map;
|
||||||
|
|
||||||
|
import javax.servlet.Servlet;
|
||||||
|
|
||||||
import org.apache.activemq.command.BrokerInfo;
|
import org.apache.activemq.command.BrokerInfo;
|
||||||
import org.apache.activemq.transport.SocketConnectorFactory;
|
import org.apache.activemq.transport.SocketConnectorFactory;
|
||||||
import org.apache.activemq.transport.WebTransportServerSupport;
|
import org.apache.activemq.transport.WebTransportServerSupport;
|
||||||
|
@ -69,11 +71,7 @@ public class WSTransportServer extends WebTransportServerSupport {
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
if (Server.getVersion().startsWith("8")) {
|
holder.setServlet(createWSServlet());
|
||||||
holder.setServlet(new org.apache.activemq.transport.ws.jetty8.WSServlet());
|
|
||||||
} else {
|
|
||||||
holder.setServlet(new org.apache.activemq.transport.ws.jetty9.WSServlet());
|
|
||||||
}
|
|
||||||
contextHandler.addServlet(holder, "/");
|
contextHandler.addServlet(holder, "/");
|
||||||
|
|
||||||
contextHandler.setAttribute("acceptListener", getAcceptListener());
|
contextHandler.setAttribute("acceptListener", getAcceptListener());
|
||||||
|
@ -99,6 +97,15 @@ public class WSTransportServer extends WebTransportServerSupport {
|
||||||
LOG.info("Listening for connections at {}", getConnectURI());
|
LOG.info("Listening for connections at {}", getConnectURI());
|
||||||
}
|
}
|
||||||
|
|
||||||
|
private Servlet createWSServlet() throws Exception {
|
||||||
|
if (Server.getVersion().startsWith("9")) {
|
||||||
|
return (Servlet)Class.forName("org.apache.activemq.transport.ws.jetty9.WSServlet", true,
|
||||||
|
getClass().getClassLoader()).newInstance();
|
||||||
|
}
|
||||||
|
return (Servlet)Class.forName("org.apache.activemq.transport.ws.jetty8.WSServlet", true,
|
||||||
|
getClass().getClassLoader()).newInstance();
|
||||||
|
}
|
||||||
|
|
||||||
private int getConnectorLocalPort() throws Exception {
|
private int getConnectorLocalPort() throws Exception {
|
||||||
return (Integer)connector.getClass().getMethod("getLocalPort").invoke(connector);
|
return (Integer)connector.getClass().getMethod("getLocalPort").invoke(connector);
|
||||||
}
|
}
|
||||||
|
|
|
@ -140,9 +140,9 @@
|
||||||
javax.management.remote,
|
javax.management.remote,
|
||||||
javax.naming,
|
javax.naming,
|
||||||
org.w3c.dom,
|
org.w3c.dom,
|
||||||
javax.servlet,
|
javax.servlet;version="[2.5,4)",
|
||||||
javax.servlet.http,
|
javax.servlet.http;version="[2.5,4)",
|
||||||
javax.servlet.resources,
|
javax.servlet.resources;version="[2.5,4)",
|
||||||
javax.servlet.jsp,
|
javax.servlet.jsp,
|
||||||
javax.servlet.jsp.tagext,
|
javax.servlet.jsp.tagext,
|
||||||
javax.servlet.jsp.el,
|
javax.servlet.jsp.el,
|
||||||
|
|
Loading…
Reference in New Issue