From a9ed4c298206fb14503811151fab0d03ad1d570e Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Mon, 22 Aug 2022 12:19:52 -0700 Subject: [PATCH] GUACAMOLE-990: Revise guacamole-auth-ban log levels to generally always notify of problematic addresses. --- .../guacamole/auth/ban/AuthenticationFailureTracker.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/extensions/guacamole-auth-ban/src/main/java/org/apache/guacamole/auth/ban/AuthenticationFailureTracker.java b/extensions/guacamole-auth-ban/src/main/java/org/apache/guacamole/auth/ban/AuthenticationFailureTracker.java index 85f5cc09c..72b4fc8dd 100644 --- a/extensions/guacamole-auth-ban/src/main/java/org/apache/guacamole/auth/ban/AuthenticationFailureTracker.java +++ b/extensions/guacamole-auth-ban/src/main/java/org/apache/guacamole/auth/ban/AuthenticationFailureTracker.java @@ -206,7 +206,7 @@ public class AuthenticationFailureTracker { AuthenticationFailureStatus status; if (failed) { status = getAuthenticationFailure(address); - logger.debug("Authentication has failed for address \"{}\" (current total failures: {}/{}).", + logger.info("Authentication has failed for address \"{}\" (current total failures: {}/{}).", address, status.getFailures(), maxAttempts); } else @@ -217,7 +217,7 @@ public class AuthenticationFailureTracker { // Explicitly block further processing of authentication/authorization // if too many failures have occurred if (status.isBlocked()) { - logger.debug("Blocking authentication attempt from address \"{}\" due to number of authentication failures.", address); + logger.warn("Blocking authentication attempt from address \"{}\" due to number of authentication failures.", address); throw new TranslatableGuacamoleClientTooManyException("Too " + "many failed authentication attempts.", "LOGIN.ERROR_TOO_MANY_ATTEMPTS");