throw ISE if WebSocketSCI.configure() is called on started ServletCon…

…textHandler Signed-off-by: Lachlan Roberts <lachlan@webtide.com>
jetty · Oct 15, 2020 · dc74bcf · dc74bcf
1 parent 5e60837
commit dc74bcf
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/...clipse/jetty/websocket/javax/server/config/JavaxWebSocketServletContainerInitializer.java b/...clipse/jetty/websocket/javax/server/config/JavaxWebSocketServletContainerInitializer.java
@@ -99,6 +99,9 @@ public interface Configurator
      */
     public static void configure(ServletContextHandler context, Configurator configurator)
     {
+        if (context.isStarted())
+            throw new IllegalStateException("configure should be called before starting");
+
         // In this embedded-jetty usage, allow ServletContext.addListener() to
         // add other ServletContextListeners (such as the ContextDestroyListener) after
         // the initialization phase is over. (important for this SCI to function)

diff --git a/.../org/eclipse/jetty/websocket/server/config/JettyWebSocketServletContainerInitializer.java b/.../org/eclipse/jetty/websocket/server/config/JettyWebSocketServletContainerInitializer.java
@@ -53,6 +53,9 @@ public interface Configurator
      */
     public static void configure(ServletContextHandler context, Configurator configurator)
     {
+        if (context.isStarted())
+            throw new IllegalStateException("configure should be called before starting");
+
         context.addEventListener(
             ContainerInitializer
                 .asContextListener(new JettyWebSocketServletContainerInitializer())