suppress warnings for rule java:S112
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 cb0a7f5..dcb3ddb 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
@@ -39,6 +39,7 @@
private RequestUtil() {
}
+ @SuppressWarnings("java:S112")
static String path(@NotNull final SlingHttpServletRequest request) throws Exception {
final String value = request.getParameter("path");
if (value == null) {
@@ -48,7 +49,7 @@
}
}
- @SuppressWarnings({"javasecurity:S2631", "checkstyle:IllegalCatch"})
+ @SuppressWarnings({"java:S112", "javasecurity:S2631", "checkstyle:IllegalCatch"})
static Pattern pattern(@NotNull final SlingHttpServletRequest request, @NotNull final Pattern defaultPattern) throws Exception {
final String value = request.getParameter("pattern"); // ensure pattern input is from trusted source, e.g. authenticated and authorized users (ClamJcrScanServlet)
if (value == null) {
@@ -62,7 +63,7 @@
}
}
- @SuppressWarnings("checkstyle:IllegalCatch")
+ @SuppressWarnings({"java:S112", "checkstyle:IllegalCatch"})
static Set<Integer> propertyTypes(@NotNull final SlingHttpServletRequest request, @NotNull final Set<Integer> defaultPropertyTypes) throws Exception {
final String[] values = request.getParameterValues("propertyTypes");
if (values == null || values.length == 0) {
@@ -75,7 +76,7 @@
}
}
- @SuppressWarnings("checkstyle:IllegalCatch")
+ @SuppressWarnings({"java:S112", "checkstyle:IllegalCatch"})
static long maxLength(@NotNull final SlingHttpServletRequest request, final long defaultMaxLength) throws Exception {
final String value = request.getParameter("maxLength");
if (value == null) {
@@ -89,7 +90,7 @@
}
}
- @SuppressWarnings("checkstyle:IllegalCatch")
+ @SuppressWarnings({"java:S112", "checkstyle:IllegalCatch"})
static int maxDepth(@NotNull final SlingHttpServletRequest request, final int defaultMaxDepth) throws Exception {
final String value = request.getParameter("maxDepth");
if (value == null) {
@@ -103,7 +104,7 @@
}
}
- @SuppressWarnings("checkstyle:ReturnCount")
+ @SuppressWarnings({"java:S112", "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) {
diff --git a/src/main/java/org/apache/sling/clam/internal/ClamUtil.java b/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
index 84e23af..35b9ca3 100644
--- a/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
+++ b/src/main/java/org/apache/sling/clam/internal/ClamUtil.java
@@ -92,6 +92,7 @@
return length <= maxLength;
}
+ @SuppressWarnings("java:S112")
public static Set<Integer> propertyTypesFromNames(@NotNull final String[] names) throws Exception {
final Set<Integer> propertyTypes = new HashSet<>();
for (final String name : names) {
diff --git a/src/main/java/org/apache/sling/clam/job/internal/JcrPropertyScanJobConsumer.java b/src/main/java/org/apache/sling/clam/job/internal/JcrPropertyScanJobConsumer.java
index a0ec82c..ee10dde 100644
--- a/src/main/java/org/apache/sling/clam/job/internal/JcrPropertyScanJobConsumer.java
+++ b/src/main/java/org/apache/sling/clam/job/internal/JcrPropertyScanJobConsumer.java
@@ -146,6 +146,7 @@
}
}
+ @SuppressWarnings("java:S112")
private InputStream getInputStream(final Value value) throws Exception {
final int propertyType = value.getType();
if (propertyType == PropertyType.BINARY) {
diff --git a/src/main/java/org/apache/sling/clam/result/internal/MailSendingScanResultHandler.java b/src/main/java/org/apache/sling/clam/result/internal/MailSendingScanResultHandler.java
index d7df2b3..9d60204 100644
--- a/src/main/java/org/apache/sling/clam/result/internal/MailSendingScanResultHandler.java
+++ b/src/main/java/org/apache/sling/clam/result/internal/MailSendingScanResultHandler.java
@@ -67,7 +67,7 @@
ocd = MailSendingScanResultHandlerConfiguration.class,
factory = true
)
-@SuppressWarnings({"java:S3077", "checkstyle:ClassFanOutComplexity"})
+@SuppressWarnings({"java:S112", "java:S3077", "checkstyle:ClassFanOutComplexity"})
public final class MailSendingScanResultHandler implements JcrPropertyScanResultHandler {
@Reference(
diff --git a/src/main/java/org/apache/sling/clam/result/internal/ResourcePersistingScanResultHandler.java b/src/main/java/org/apache/sling/clam/result/internal/ResourcePersistingScanResultHandler.java
index 4d1cfec..fa511b9 100644
--- a/src/main/java/org/apache/sling/clam/result/internal/ResourcePersistingScanResultHandler.java
+++ b/src/main/java/org/apache/sling/clam/result/internal/ResourcePersistingScanResultHandler.java
@@ -143,6 +143,7 @@
return !scanResult.isOk() || configuration.result_status_ok_persist();
}
+ @SuppressWarnings("java:S112")
private void persistResult(@NotNull final ScanResult scanResult, @NotNull final String path, final Integer index, final int propertyType, @Nullable final String userId) {
try (ResourceResolver resourceResolver = serviceResourceResolver()) {
final Map<String, Object> properties = properties(path, index, userId, scanResult);