GUACAMOLE-197: Log error if we reach an unexpected place in the RADIUS conversation.
Project: http://git-wip-us.apache.org/repos/asf/guacamole-client/repo Commit: http://git-wip-us.apache.org/repos/asf/guacamole-client/commit/41557566 Tree: http://git-wip-us.apache.org/repos/asf/guacamole-client/tree/41557566 Diff: http://git-wip-us.apache.org/repos/asf/guacamole-client/diff/41557566 Branch: refs/heads/master Commit: 41557566b9b87b7f2acaad00fc54781e83210658 Parents: 77cb98c Author: Nick Couchman <vn...@apache.org> Authored: Fri Jul 14 16:53:39 2017 -0400 Committer: Nick Couchman <vn...@apache.org> Committed: Mon Jan 29 17:08:11 2018 -0500 ---------------------------------------------------------------------- .../guacamole/auth/radius/AuthenticationProviderService.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/guacamole-client/blob/41557566/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java ---------------------------------------------------------------------- diff --git a/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java b/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java index da0ae4e..d4f3ed1 100644 --- a/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java +++ b/extensions/guacamole-auth-radius/src/main/java/org/apache/guacamole/auth/radius/AuthenticationProviderService.java @@ -182,8 +182,10 @@ public class AuthenticationProviderService { } // Something unanticipated happened, so we panic - else + else { + logger.error("Unexpected authentication failure talking to RADIUS server."); throw new GuacamoleInvalidCredentialsException("Unknown error trying to authenticate.", CredentialsInfo.USERNAME_PASSWORD); + } } // The user responded to the challenge, send that back to the server