GUACAMOLE-2181: Include full exception when logging at debug level.
This commit is contained in:
1 file changed
+9
-6
@@ -183,30 +183,33 @@ public class EventLoggingListener implements Listener {
|
|||||||
if (creds.isEmpty())
|
if (creds.isEmpty())
|
||||||
logger.debug("Empty authentication attempt (login screen "
|
logger.debug("Empty authentication attempt (login screen "
|
||||||
+ "initialization) from {} failed: {}",
|
+ "initialization) from {} failed: {}",
|
||||||
new RemoteAddress(creds), new Failure(event));
|
new RemoteAddress(creds), new Failure(event), event.getFailure());
|
||||||
else if (username == null || username.isEmpty())
|
else if (username == null || username.isEmpty())
|
||||||
logger.debug("Anonymous authentication attempt from {} failed: {}",
|
logger.debug("Anonymous authentication attempt from {} failed: {}",
|
||||||
new RemoteAddress(creds), new Failure(event));
|
new RemoteAddress(creds), new Failure(event), event.getFailure());
|
||||||
else if (event.getFailure() instanceof GuacamoleInsufficientCredentialsException) {
|
else if (event.getFailure() instanceof GuacamoleInsufficientCredentialsException) {
|
||||||
if (authProvider != null)
|
if (authProvider != null)
|
||||||
logger.debug("Authentication attempt from {} for user \"{}\" "
|
logger.debug("Authentication attempt from {} for user \"{}\" "
|
||||||
+ "requires additional credentials to continue: {} "
|
+ "requires additional credentials to continue: {} "
|
||||||
+ "(requested by \"{}\")", new RemoteAddress(creds),
|
+ "(requested by \"{}\")", new RemoteAddress(creds),
|
||||||
username, new Failure(event), authProvider.getIdentifier());
|
username, new Failure(event), authProvider.getIdentifier(),
|
||||||
|
event.getFailure());
|
||||||
else
|
else
|
||||||
logger.debug("Authentication attempt from {} for user \"{}\" "
|
logger.debug("Authentication attempt from {} for user \"{}\" "
|
||||||
+ "requires additional credentials to continue: {}",
|
+ "requires additional credentials to continue: {}",
|
||||||
new RemoteAddress(creds), username, new Failure(event));
|
new RemoteAddress(creds), username, new Failure(event),
|
||||||
|
event.getFailure());
|
||||||
}
|
}
|
||||||
else {
|
else {
|
||||||
if (authProvider != null)
|
if (authProvider != null)
|
||||||
logger.warn("Authentication attempt from {} for user \"{}\" "
|
logger.warn("Authentication attempt from {} for user \"{}\" "
|
||||||
+ "failed: {} (rejected by \"{}\")", new RemoteAddress(creds),
|
+ "failed: {} (rejected by \"{}\")", new RemoteAddress(creds),
|
||||||
username, new Failure(event), authProvider.getIdentifier());
|
username, new Failure(event), authProvider.getIdentifier(),
|
||||||
|
event.getFailure());
|
||||||
else
|
else
|
||||||
logger.warn("Authentication attempt from {} for user \"{}\" "
|
logger.warn("Authentication attempt from {} for user \"{}\" "
|
||||||
+ "failed: {}", new RemoteAddress(creds), username,
|
+ "failed: {}", new RemoteAddress(creds), username,
|
||||||
new Failure(event));
|
new Failure(event), event.getFailure());
|
||||||
}
|
}
|
||||||
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in new issue
Block a user