Merge pull request #5491 from eclipse/jetty-9.4.x-5488-jetty-dir.css

Fixes #5488 - jetty-dir.css not found when using JPMS.

Author: sbordet

jetty-server/src/main/java/org/eclipse/jetty/server/handler/ResourceHandler.java

@@ -226,12 +226,17 @@ public Resource getStylesheet()
         {
             if (_defaultStylesheet == null)
             {
-                _defaultStylesheet = Resource.newResource(this.getClass().getResource("/jetty-dir.css"));
+                _defaultStylesheet = getDefaultStylesheet();
             }
             return _defaultStylesheet;
         }
     }
 
+    public static Resource getDefaultStylesheet()
+    {
+        return Resource.newResource(ResourceHandler.class.getResource("/jetty-dir.css"));
+    }
+
     public String[] getWelcomeFiles()
     {
         return _welcomes;

jetty-util/src/main/resources/jetty-dir.css renamed to jetty-server/src/main/resources/jetty-dir.css

@@ -41,6 +41,7 @@
 import org.eclipse.jetty.server.ResourceService;
 import org.eclipse.jetty.server.ResourceService.WelcomeFactory;
 import org.eclipse.jetty.server.handler.ContextHandler;
+import org.eclipse.jetty.server.handler.ResourceHandler;
 import org.eclipse.jetty.util.URIUtil;
 import org.eclipse.jetty.util.log.Log;
 import org.eclipse.jetty.util.log.Logger;
@@ -222,7 +223,7 @@ public void init()
             }
             if (_stylesheet == null)
             {
-                _stylesheet = Resource.newResource(this.getClass().getResource("/jetty-dir.css"));
+                _stylesheet = ResourceHandler.getDefaultStylesheet();
             }
         }
         catch (Exception e)

jetty-servlet/src/main/java/org/eclipse/jetty/servlet/DefaultServlet.java

@@ -202,8 +202,11 @@ public void testJPMS() throws Exception
             assertTrue(run.awaitConsoleLogsFor("Started @", 10, TimeUnit.SECONDS));
 
             startHttpClient();
-            ContentResponse response = client.GET("http://localhost:" + httpPort + "/test/hello");
-            assertEquals(HttpStatus.OK_200, response.getStatus());
+            ContentResponse helloResponse = client.GET("http://localhost:" + httpPort + "/test/hello");
+            assertEquals(HttpStatus.OK_200, helloResponse.getStatus());
+
+            ContentResponse cssResponse = client.GET("http://localhost:" + httpPort + "/jetty-dir.css");
+            assertEquals(HttpStatus.OK_200, cssResponse.getStatus());
         }
     }