Issue #6026 - Cleaning up uncovered HTTP methods warning

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
jetty · Mar 3, 2021 · 82a5072 · 82a5072
1 parent 1cb0a49
commit 82a5072
Showing 1 changed file with 5 additions and 10 deletions.
diff --git a/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java b/jetty-security/src/main/java/org/eclipse/jetty/security/ConstraintSecurityHandler.java
@@ -716,8 +716,8 @@ protected boolean checkWebResourcePermissions(String pathInContext, Request requ
     public void dump(Appendable out, String indent) throws IOException
     {
         dumpObjects(out, indent,
-                    DumpableCollection.from("roles", _roles),
-                    DumpableCollection.from("constraints", _constraintMappings));
+            DumpableCollection.from("roles", _roles),
+            DumpableCollection.from("constraints", _constraintMappings));
     }
 
     @Override
@@ -741,14 +741,9 @@ public boolean checkPathsWithUncoveredHttpMethods()
         Set<String> paths = getPathsWithUncoveredHttpMethods();
         if (paths != null && !paths.isEmpty())
         {
-            ContextHandler.Context currentContext = ContextHandler.getCurrentContext();
-
-            for (String p : paths)
-            {
-                LOG.warn("{} has uncovered http methods for path: {}", currentContext, p);
-            }
-            if (LOG.isDebugEnabled())
-                LOG.debug("{} has uncovered http methods", currentContext, new Throwable());
+            LOG.warn("{} has paths with uncovered http methods: [{}]",
+                ContextHandler.getCurrentContext(),
+                String.join(", ", paths));
             return true;
         }
         return false;