GUACAMOLE-197: Brush up log messages and change levels for invalid state information.

This commit is contained in:
Nick Couchman
2018-02-04 09:31:47 -05:00
parent ef6d0c6a53
commit b9a6880470

View File

@@ -160,8 +160,7 @@ public class AuthenticationProviderService {
try { try {
String stateString = request.getParameter(RadiusStateField.PARAMETER_NAME); String stateString = request.getParameter(RadiusStateField.PARAMETER_NAME);
if (stateString == null) { if (stateString == null) {
logger.error("Could not retrieve RADIUS state."); logger.warn("Expected state parameter was not present in challenge/response.");
logger.debug("Received null value while retrieving RADIUS state parameter.");
throw new GuacamoleInvalidCredentialsException("Authentication error.", CredentialsInfo.USERNAME_PASSWORD); throw new GuacamoleInvalidCredentialsException("Authentication error.", CredentialsInfo.USERNAME_PASSWORD);
} }
@@ -171,8 +170,8 @@ public class AuthenticationProviderService {
stateBytes); stateBytes);
} }
catch (IllegalArgumentException e) { catch (IllegalArgumentException e) {
logger.error("Illegal argument while parsing RADIUS state string.", e.getMessage()); logger.warn("Illegal hexadecimal value while parsing RADIUS state string.", e.getMessage());
logger.debug("Illegal argument found while parsing the RADIUS state string.", e); logger.debug("Encountered exception while attepmting to perse the hexidecimanl state value.", e);
throw new GuacamoleInvalidCredentialsException("Authentication error.", CredentialsInfo.USERNAME_PASSWORD); throw new GuacamoleInvalidCredentialsException("Authentication error.", CredentialsInfo.USERNAME_PASSWORD);
} }
catch (GuacamoleException e) { catch (GuacamoleException e) {