GUACAMOLE-611: Update internal failure warnings to reflect the fact that an auth attempt is being ignored, not "denied".
Project: http://git-wip-us.apache.org/repos/asf/guacamole-client/repo Commit: http://git-wip-us.apache.org/repos/asf/guacamole-client/commit/d28c4a7c Tree: http://git-wip-us.apache.org/repos/asf/guacamole-client/tree/d28c4a7c Diff: http://git-wip-us.apache.org/repos/asf/guacamole-client/diff/d28c4a7c Branch: refs/heads/master Commit: d28c4a7cce76315f7738d8f8229cbc42f2944497 Parents: 9d4aae1 Author: Michael Jumper <mjum...@apache.org> Authored: Fri Sep 7 12:36:11 2018 -0700 Committer: Michael Jumper <mjum...@apache.org> Committed: Fri Sep 7 12:36:11 2018 -0700 ---------------------------------------------------------------------- .../extension/AuthenticationProviderFacade.java | 16 ++++++++++++---- 1 file changed, 12 insertions(+), 4 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/d28c4a7c/guacamole/src/main/java/org/apache/guacamole/extension/AuthenticationProviderFacade.java ---------------------------------------------------------------------- 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 2b4f99c..6c6474b 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; }