Issue #6026 - Cleaning up uncovered HTTP methods warning

Signed-off-by: Joakim Erdfelt <joakim.erdfelt@gmail.com>
This commit is contained in:
Joakim Erdfelt 2021-03-03 14:40:46 -06:00
parent 1cb0a4978f
commit 82a50727d8
No known key found for this signature in database
GPG Key ID: 2D0E1FB8FE4B68B4
1 changed files with 5 additions and 10 deletions

View File

@ -716,8 +716,8 @@ public class ConstraintSecurityHandler extends SecurityHandler implements Constr
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 class ConstraintSecurityHandler extends SecurityHandler implements Constr
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;