Review log levels (#1015)

acoburn · acoburn · commit f7b47f1b8056 · 2020-08-13T22:41:24.000-04:00
diff --git a/auth/basic/src/main/java/org/trellisldp/auth/basic/Credentials.java b/auth/basic/src/main/java/org/trellisldp/auth/basic/Credentials.java
@@ -67,7 +67,7 @@ public static Credentials parse(final String encoded) {
                 return new Credentials(parts[0], parts[1]);
             }
         } catch (final IllegalArgumentException ex) {
-            LOGGER.warn("Invalid credentials provided: {}", ex.getMessage());
+            LOGGER.debug("Invalid credentials provided: {}", ex.getMessage());
         }
         return null;
     }
diff --git a/auth/oauth/src/main/java/org/trellisldp/oauth/OAuthFilter.java b/auth/oauth/src/main/java/org/trellisldp/oauth/OAuthFilter.java
@@ -129,7 +129,7 @@ private Principal authenticate(final String token) {
         } catch (final SecurityException ex) {
             LOGGER.debug("Invalid signature, ignoring JWT token: {}", ex.getMessage());
         } catch (final JwtException ex) {
-            LOGGER.warn("Problem reading JWT value: {}", ex.getMessage());
+            LOGGER.debug("Problem reading JWT value: {}", ex.getMessage());
         }
         return null;
     }
diff --git a/auth/webac/src/main/java/org/trellisldp/webac/WebAcFilter.java b/auth/webac/src/main/java/org/trellisldp/webac/WebAcFilter.java
@@ -250,50 +250,50 @@ public void filter(final ContainerRequestContext req, final ContainerResponseCon
 
     protected void verifyCanAppend(final Set<IRI> modes, final Session session, final String path) {
         if (!modes.contains(ACL.Append) && !modes.contains(ACL.Write)) {
-            LOGGER.warn("User: {} cannot Append to {}", session.getAgent(), path);
+            LOGGER.debug("User: {} cannot Append to {}", session.getAgent(), path);
             if (AnonymousAgent.equals(session.getAgent())) {
                 throw new NotAuthorizedException(challenges.get(0),
                         challenges.subList(1, challenges.size()).toArray());
             }
             throw new ForbiddenException();
         }
-        LOGGER.debug("User: {} can append to {}", session.getAgent(), path);
+        LOGGER.trace("User: {} can append to {}", session.getAgent(), path);
     }
 
     protected void verifyCanControl(final Set<IRI> modes, final Session session, final String path) {
         if (!modes.contains(ACL.Control)) {
-            LOGGER.warn("User: {} cannot Control {}", session.getAgent(), path);
+            LOGGER.debug("User: {} cannot Control {}", session.getAgent(), path);
             if (AnonymousAgent.equals(session.getAgent())) {
                 throw new NotAuthorizedException(challenges.get(0),
                         challenges.subList(1, challenges.size()).toArray());
             }
             throw new ForbiddenException();
         }
-        LOGGER.debug("User: {} can control {}", session.getAgent(), path);
+        LOGGER.trace("User: {} can control {}", session.getAgent(), path);
     }
 
     protected void verifyCanWrite(final Set<IRI> modes, final Session session, final String path) {
         if (!modes.contains(ACL.Write)) {
-            LOGGER.warn("User: {} cannot Write to {}", session.getAgent(), path);
+            LOGGER.debug("User: {} cannot Write to {}", session.getAgent(), path);
             if (AnonymousAgent.equals(session.getAgent())) {
                 throw new NotAuthorizedException(challenges.get(0),
                         challenges.subList(1, challenges.size()).toArray());
             }
             throw new ForbiddenException();
         }
-        LOGGER.debug("User: {} can write to {}", session.getAgent(), path);
+        LOGGER.trace("User: {} can write to {}", session.getAgent(), path);
     }
 
     protected void verifyCanRead(final Set<IRI> modes, final Session session, final String path) {
         if (!modes.contains(ACL.Read)) {
-            LOGGER.warn("User: {} cannot Read from {}", session.getAgent(), path);
+            LOGGER.debug("User: {} cannot Read from {}", session.getAgent(), path);
             if (AnonymousAgent.equals(session.getAgent())) {
                 throw new NotAuthorizedException(challenges.get(0),
                         challenges.subList(1, challenges.size()).toArray());
             }
             throw new ForbiddenException();
         }
-        LOGGER.debug("User: {} can read {}", session.getAgent(), path);
+        LOGGER.trace("User: {} can read {}", session.getAgent(), path);
     }
 
     static boolean reqAudit(final Prefer prefer) {
diff --git a/components/file/src/main/java/org/trellisldp/file/FileNamespaceService.java b/components/file/src/main/java/org/trellisldp/file/FileNamespaceService.java
@@ -82,7 +82,7 @@ public boolean setPrefix(final String prefix, final String namespace) {
         requireNonNull(namespace, "The namespce value may not be null!");
 
         if (dataRev.containsKey(namespace)) {
-            LOGGER.warn("A prefix already exists for the namespace: {}", namespace);
+            LOGGER.debug("A prefix already exists for the namespace: {}", namespace);
             return false;
         }
 
diff --git a/components/file/src/main/java/org/trellisldp/file/FileUtils.java b/components/file/src/main/java/org/trellisldp/file/FileUtils.java
@@ -131,7 +131,7 @@ public static Stream<Quad> parseQuad(final String line) {
         } else if (nodes.size() == 4) {
             return of(fromJena(create(nodes.get(3), nodes.get(0), nodes.get(1), nodes.get(2))));
         } else {
-            LOGGER.warn("Skipping invalid data value: {}", line);
+            LOGGER.debug("Skipping invalid data value: {}", line);
             return empty();
         }
     }
diff --git a/components/jdbc/src/main/java/org/trellisldp/jdbc/DBNamespaceService.java b/components/jdbc/src/main/java/org/trellisldp/jdbc/DBNamespaceService.java
@@ -86,7 +86,7 @@ public boolean setPrefix(final String prefix, final String namespace) {
                     handle.execute("INSERT INTO namespaces (prefix, namespace) VALUES (?, ?)", prefix, namespace));
                 return true;
             } catch (final StatementException ex) {
-                LOGGER.warn("Could not save prefix {} with namespace {}: {}", prefix, namespace, ex.getMessage());
+                LOGGER.debug("Could not save prefix {} with namespace {}: {}", prefix, namespace, ex.getMessage());
             }
         }
         return false;
diff --git a/core/common/src/main/java/org/trellisldp/common/AcceptDatetime.java b/core/common/src/main/java/org/trellisldp/common/AcceptDatetime.java
@@ -78,7 +78,7 @@ private static Instant parseDatetime(final String datetime) {
         try {
             return parse(datetime.trim(), RFC_1123_DATE_TIME).toInstant();
         } catch (final DateTimeException ex) {
-            LOGGER.warn("Invalid date supplied ({}): {}", datetime, ex.getMessage());
+            LOGGER.debug("Invalid date supplied ({}): {}", datetime, ex.getMessage());
         }
         return null;
     }
diff --git a/core/common/src/main/java/org/trellisldp/common/Forwarded.java b/core/common/src/main/java/org/trellisldp/common/Forwarded.java
@@ -99,7 +99,7 @@ public OptionalInt getPort() {
             try {
                 return OptionalInt.of(Integer.parseInt(port));
             } catch (final NumberFormatException ex) {
-                LOGGER.warn("Could not parse port number: {}", ex.getMessage());
+                LOGGER.debug("Could not parse port number: {}", ex.getMessage());
             }
         }
         return OptionalInt.empty();
diff --git a/core/common/src/main/java/org/trellisldp/common/Range.java b/core/common/src/main/java/org/trellisldp/common/Range.java
@@ -88,7 +88,7 @@ private static int[] parse(final String range) {
                         return ints;
                     }
                 } catch (final NumberFormatException ex) {
-                    LOGGER.warn("Invalid Range request: {}", ex.getMessage());
+                    LOGGER.debug("Invalid Range request: {}", ex.getMessage());
                 }
             }
         }
diff --git a/core/common/src/main/java/org/trellisldp/common/Slug.java b/core/common/src/main/java/org/trellisldp/common/Slug.java
@@ -74,7 +74,7 @@ private static String decodeSlug(final String value) {
         try {
             return DECODER.decode(value);
         } catch (final DecoderException ex) {
-            LOGGER.warn("Error decoding slug value, ignoring header: {}", ex.getMessage());
+            LOGGER.debug("Error decoding slug value, ignoring header: {}", ex.getMessage());
         }
         return null;
     }
diff --git a/core/common/src/main/java/org/trellisldp/common/Version.java b/core/common/src/main/java/org/trellisldp/common/Version.java
@@ -61,7 +61,7 @@ private static Instant parse(final String version) {
         try {
             return ofEpochSecond(parseLong(version.trim()));
         } catch (final NumberFormatException ex) {
-            LOGGER.warn("Unable to parse version string '{}': {}", version, ex.getMessage());
+            LOGGER.debug("Unable to parse version string '{}': {}", version, ex.getMessage());
         }
         return null;
     }
diff --git a/core/http/src/main/java/org/trellisldp/http/impl/PatchHandler.java b/core/http/src/main/java/org/trellisldp/http/impl/PatchHandler.java
@@ -139,7 +139,7 @@ public ResponseBuilder initialize(final Resource parent, final Resource resource
                 .entity("Unsupported interaction model provided").type(TEXT_PLAIN_TYPE).build());
         } else if (syntax == null) {
             // Get the incoming syntax and check that the underlying I/O service supports it
-            LOGGER.warn("Content-Type: {} not supported", getRequest().getContentType());
+            LOGGER.debug("Content-Type: {} not supported", getRequest().getContentType());
             throw new NotSupportedException();
         }
         // Check the cache headers
diff --git a/core/http/src/main/java/org/trellisldp/http/impl/PostHandler.java b/core/http/src/main/java/org/trellisldp/http/impl/PostHandler.java
@@ -208,7 +208,6 @@ private CompletionStage<ResponseBuilder> handleResourceCreation(final Dataset mu
 
         getAuditQuadData().forEachOrdered(immutable::add);
 
-        LOGGER.info("Creating resource");
         return handleResourceCreation(metadata, mutable, immutable)
             .thenCompose(future -> persistBinaryContent(binary))
             .thenCompose(future -> emitEvent(internalId, AS.Create, ldpType))

Original file line number	Diff line number	Diff line change
`@@ -67,7 +67,7 @@ public static Credentials parse(final String encoded) {`
`67`	`67`	`return new Credentials(parts[0], parts[1]);`
`68`	`68`	`}`
`69`	`69`	`} catch (final IllegalArgumentException ex) {`
`70`		`- LOGGER.warn("Invalid credentials provided: {}", ex.getMessage());`
	`70`	`+ LOGGER.debug("Invalid credentials provided: {}", ex.getMessage());`
`71`	`71`	`}`
`72`	`72`	`return null;`
`73`	`73`	`}`
Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ private Principal authenticate(final String token) {`
`129`	`129`	`} catch (final SecurityException ex) {`
`130`	`130`	`LOGGER.debug("Invalid signature, ignoring JWT token: {}", ex.getMessage());`
`131`	`131`	`} catch (final JwtException ex) {`
`132`		`- LOGGER.warn("Problem reading JWT value: {}", ex.getMessage());`
	`132`	`+ LOGGER.debug("Problem reading JWT value: {}", ex.getMessage());`
`133`	`133`	`}`
`134`	`134`	`return null;`
`135`	`135`	`}`
Original file line number	Diff line number	Diff line change
`@@ -250,50 +250,50 @@ public void filter(final ContainerRequestContext req, final ContainerResponseCon`
`250`	`250`
`251`	`251`	`protected void verifyCanAppend(final Set<IRI> modes, final Session session, final String path) {`
`252`	`252`	`if (!modes.contains(ACL.Append) && !modes.contains(ACL.Write)) {`
`253`		`- LOGGER.warn("User: {} cannot Append to {}", session.getAgent(), path);`
	`253`	`+ LOGGER.debug("User: {} cannot Append to {}", session.getAgent(), path);`
`254`	`254`	`if (AnonymousAgent.equals(session.getAgent())) {`
`255`	`255`	`throw new NotAuthorizedException(challenges.get(0),`
`256`	`256`	`challenges.subList(1, challenges.size()).toArray());`
`257`	`257`	`}`
`258`	`258`	`throw new ForbiddenException();`
`259`	`259`	`}`
`260`		`- LOGGER.debug("User: {} can append to {}", session.getAgent(), path);`
	`260`	`+ LOGGER.trace("User: {} can append to {}", session.getAgent(), path);`
`261`	`261`	`}`
`262`	`262`
`263`	`263`	`protected void verifyCanControl(final Set<IRI> modes, final Session session, final String path) {`
`264`	`264`	`if (!modes.contains(ACL.Control)) {`
`265`		`- LOGGER.warn("User: {} cannot Control {}", session.getAgent(), path);`
	`265`	`+ LOGGER.debug("User: {} cannot Control {}", session.getAgent(), path);`
`266`	`266`	`if (AnonymousAgent.equals(session.getAgent())) {`
`267`	`267`	`throw new NotAuthorizedException(challenges.get(0),`
`268`	`268`	`challenges.subList(1, challenges.size()).toArray());`
`269`	`269`	`}`
`270`	`270`	`throw new ForbiddenException();`
`271`	`271`	`}`
`272`		`- LOGGER.debug("User: {} can control {}", session.getAgent(), path);`
	`272`	`+ LOGGER.trace("User: {} can control {}", session.getAgent(), path);`
`273`	`273`	`}`
`274`	`274`
`275`	`275`	`protected void verifyCanWrite(final Set<IRI> modes, final Session session, final String path) {`
`276`	`276`	`if (!modes.contains(ACL.Write)) {`
`277`		`- LOGGER.warn("User: {} cannot Write to {}", session.getAgent(), path);`
	`277`	`+ LOGGER.debug("User: {} cannot Write to {}", session.getAgent(), path);`
`278`	`278`	`if (AnonymousAgent.equals(session.getAgent())) {`
`279`	`279`	`throw new NotAuthorizedException(challenges.get(0),`
`280`	`280`	`challenges.subList(1, challenges.size()).toArray());`
`281`	`281`	`}`
`282`	`282`	`throw new ForbiddenException();`
`283`	`283`	`}`
`284`		`- LOGGER.debug("User: {} can write to {}", session.getAgent(), path);`
	`284`	`+ LOGGER.trace("User: {} can write to {}", session.getAgent(), path);`
`285`	`285`	`}`
`286`	`286`
`287`	`287`	`protected void verifyCanRead(final Set<IRI> modes, final Session session, final String path) {`
`288`	`288`	`if (!modes.contains(ACL.Read)) {`
`289`		`- LOGGER.warn("User: {} cannot Read from {}", session.getAgent(), path);`
	`289`	`+ LOGGER.debug("User: {} cannot Read from {}", session.getAgent(), path);`
`290`	`290`	`if (AnonymousAgent.equals(session.getAgent())) {`
`291`	`291`	`throw new NotAuthorizedException(challenges.get(0),`
`292`	`292`	`challenges.subList(1, challenges.size()).toArray());`
`293`	`293`	`}`
`294`	`294`	`throw new ForbiddenException();`
`295`	`295`	`}`
`296`		`- LOGGER.debug("User: {} can read {}", session.getAgent(), path);`
	`296`	`+ LOGGER.trace("User: {} can read {}", session.getAgent(), path);`
`297`	`297`	`}`
`298`	`298`
`299`	`299`	`static boolean reqAudit(final Prefer prefer) {`
Original file line number	Diff line number	Diff line change
`@@ -82,7 +82,7 @@ public boolean setPrefix(final String prefix, final String namespace) {`
`82`	`82`	`requireNonNull(namespace, "The namespce value may not be null!");`
`83`	`83`
`84`	`84`	`if (dataRev.containsKey(namespace)) {`
`85`		`- LOGGER.warn("A prefix already exists for the namespace: {}", namespace);`
	`85`	`+ LOGGER.debug("A prefix already exists for the namespace: {}", namespace);`
`86`	`86`	`return false;`
`87`	`87`	`}`
`88`	`88`
Original file line number	Diff line number	Diff line change
`@@ -131,7 +131,7 @@ public static Stream<Quad> parseQuad(final String line) {`
`131`	`131`	`} else if (nodes.size() == 4) {`
`132`	`132`	`return of(fromJena(create(nodes.get(3), nodes.get(0), nodes.get(1), nodes.get(2))));`
`133`	`133`	`} else {`
`134`		`- LOGGER.warn("Skipping invalid data value: {}", line);`
	`134`	`+ LOGGER.debug("Skipping invalid data value: {}", line);`
`135`	`135`	`return empty();`
`136`	`136`	`}`
`137`	`137`	`}`
Original file line number	Diff line number	Diff line change
`@@ -86,7 +86,7 @@ public boolean setPrefix(final String prefix, final String namespace) {`
`86`	`86`	`handle.execute("INSERT INTO namespaces (prefix, namespace) VALUES (?, ?)", prefix, namespace));`
`87`	`87`	`return true;`
`88`	`88`	`} catch (final StatementException ex) {`
`89`		`- LOGGER.warn("Could not save prefix {} with namespace {}: {}", prefix, namespace, ex.getMessage());`
	`89`	`+ LOGGER.debug("Could not save prefix {} with namespace {}: {}", prefix, namespace, ex.getMessage());`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`	`return false;`
Original file line number	Diff line number	Diff line change
`@@ -78,7 +78,7 @@ private static Instant parseDatetime(final String datetime) {`
`78`	`78`	`try {`
`79`	`79`	`return parse(datetime.trim(), RFC_1123_DATE_TIME).toInstant();`
`80`	`80`	`} catch (final DateTimeException ex) {`
`81`		`- LOGGER.warn("Invalid date supplied ({}): {}", datetime, ex.getMessage());`
	`81`	`+ LOGGER.debug("Invalid date supplied ({}): {}", datetime, ex.getMessage());`
`82`	`82`	`}`
`83`	`83`	`return null;`
`84`	`84`	`}`
Original file line number	Diff line number	Diff line change
`@@ -99,7 +99,7 @@ public OptionalInt getPort() {`
`99`	`99`	`try {`
`100`	`100`	`return OptionalInt.of(Integer.parseInt(port));`
`101`	`101`	`} catch (final NumberFormatException ex) {`
`102`		`- LOGGER.warn("Could not parse port number: {}", ex.getMessage());`
	`102`	`+ LOGGER.debug("Could not parse port number: {}", ex.getMessage());`
`103`	`103`	`}`
`104`	`104`	`}`
`105`	`105`	`return OptionalInt.empty();`
Original file line number	Diff line number	Diff line change
`@@ -88,7 +88,7 @@ private static int[] parse(final String range) {`
`88`	`88`	`return ints;`
`89`	`89`	`}`
`90`	`90`	`} catch (final NumberFormatException ex) {`
`91`		`- LOGGER.warn("Invalid Range request: {}", ex.getMessage());`
	`91`	`+ LOGGER.debug("Invalid Range request: {}", ex.getMessage());`
`92`	`92`	`}`
`93`	`93`	`}`
`94`	`94`	`}`
Original file line number	Diff line number	Diff line change
`@@ -74,7 +74,7 @@ private static String decodeSlug(final String value) {`
`74`	`74`	`try {`
`75`	`75`	`return DECODER.decode(value);`
`76`	`76`	`} catch (final DecoderException ex) {`
`77`		`- LOGGER.warn("Error decoding slug value, ignoring header: {}", ex.getMessage());`
	`77`	`+ LOGGER.debug("Error decoding slug value, ignoring header: {}", ex.getMessage());`
`78`	`78`	`}`
`79`	`79`	`return null;`
`80`	`80`	`}`