Skip to content

Commit

Permalink
GUACAMOLE-990: Revise guacamole-auth-ban log levels to generally alwa…
Browse files Browse the repository at this point in the history
…ys notify of problematic addresses.
  • Loading branch information
mike-jumper committed Aug 22, 2022
1 parent 5dea46c commit fe56df7
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -206,7 +206,7 @@ private void notifyAuthenticationStatus(Credentials credentials,
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
Expand All @@ -217,7 +217,7 @@ private void notifyAuthenticationStatus(Credentials credentials,
// 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");
Expand Down

0 comments on commit fe56df7

Please sign in to comment.