This is an automated email from the ASF dual-hosted git repository. olli pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-clam.git
commit 5198e8c71fc99f9368ac8ba498de5b759536b5e6 Author: Oliver Lietz <[email protected]> AuthorDate: Sun Mar 27 15:20:37 2022 +0200 [checkstyle] (coding) ReturnCount --- .../java/org/apache/sling/clam/http/internal/ClamJcrScanServlet.java | 2 +- src/main/java/org/apache/sling/clam/http/internal/RequestUtil.java | 1 + 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/src/main/java/org/apache/sling/clam/http/internal/ClamJcrScanServlet.java b/src/main/java/org/apache/sling/clam/http/internal/ClamJcrScanServlet.java index e61229f..6055244 100644 --- a/src/main/java/org/apache/sling/clam/http/internal/ClamJcrScanServlet.java +++ b/src/main/java/org/apache/sling/clam/http/internal/ClamJcrScanServlet.java @@ -117,7 +117,7 @@ public final class ClamJcrScanServlet extends SlingAllMethodsServlet { } @Override - @SuppressWarnings("checkstyle:IllegalCatch") + @SuppressWarnings({"checkstyle:IllegalCatch", "checkstyle:ReturnCount"}) protected void doPost(@NotNull final SlingHttpServletRequest request, @NotNull final SlingHttpServletResponse response) throws ServletException, IOException { final List<String> groups = Arrays.asList(configuration.scan_authorized_groups()); if (!groups.isEmpty()) { diff --git a/src/main/java/org/apache/sling/clam/http/internal/RequestUtil.java b/src/main/java/org/apache/sling/clam/http/internal/RequestUtil.java index 094bc7c..e5194b4 100644 --- a/src/main/java/org/apache/sling/clam/http/internal/RequestUtil.java +++ b/src/main/java/org/apache/sling/clam/http/internal/RequestUtil.java @@ -103,6 +103,7 @@ public final class RequestUtil { } } + @SuppressWarnings("checkstyle:ReturnCount") static boolean isAuthorized(@NotNull final SlingHttpServletRequest request, @NotNull final Collection<String> authorizedGroups) throws Exception { final Authorizable authorizable = request.getResourceResolver().adaptTo(Authorizable.class); if (authorizable == null) {
