From d28c4a7cce76315f7738d8f8229cbc42f2944497 Mon Sep 17 00:00:00 2001 From: Michael Jumper Date: Fri, 7 Sep 2018 12:36:11 -0700 Subject: [PATCH] GUACAMOLE-611: Update internal failure warnings to reflect the fact that an auth attempt is being ignored, not "denied". --- .../extension/AuthenticationProviderFacade.java | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) diff --git a/guacamole/src/main/java/org/apache/guacamole/extension/AuthenticationProviderFacade.java b/guacamole/src/main/java/org/apache/guacamole/extension/AuthenticationProviderFacade.java index 2b4f99c82..6c6474b45 100644 --- a/guacamole/src/main/java/org/apache/guacamole/extension/AuthenticationProviderFacade.java +++ b/guacamole/src/main/java/org/apache/guacamole/extension/AuthenticationProviderFacade.java @@ -179,7 +179,9 @@ public class AuthenticationProviderFacade implements AuthenticationProvider { // Ignore auth attempts if no auth provider could be loaded if (authProvider == null) { - logger.warn("Authentication attempt denied because the authentication system could not be loaded. Please check for errors earlier in the logs."); + logger.warn("Authentication attempt ignored because the relevant " + + "authentication provider could not be loaded. Please " + + "check for errors earlier in the logs."); return null; } @@ -244,7 +246,9 @@ public class AuthenticationProviderFacade implements AuthenticationProvider { // Ignore auth attempts if no auth provider could be loaded if (authProvider == null) { - logger.warn("Reauthentication attempt denied because the authentication system could not be loaded. Please check for errors earlier in the logs."); + logger.warn("Reauthentication attempt ignored because the relevant " + + "authentication provider could not be loaded. Please " + + "check for errors earlier in the logs."); return null; } @@ -259,7 +263,9 @@ public class AuthenticationProviderFacade implements AuthenticationProvider { // Ignore auth attempts if no auth provider could be loaded if (authProvider == null) { - logger.warn("User data retrieval attempt denied because the authentication system could not be loaded. Please check for errors earlier in the logs."); + logger.warn("User data retrieval attempt ignored because the " + + "relevant authentication provider could not be loaded. " + + "Please check for errors earlier in the logs."); return null; } @@ -325,7 +331,9 @@ public class AuthenticationProviderFacade implements AuthenticationProvider { // Ignore auth attempts if no auth provider could be loaded if (authProvider == null) { - logger.warn("User data refresh attempt denied because the authentication system could not be loaded. Please check for errors earlier in the logs."); + logger.warn("User data refresh attempt ignored because the " + + "relevant authentication provider could not be loaded. " + + "Please check for errors earlier in the logs."); return null; }