necouchman commented on code in PR #1028: URL: https://github.com/apache/guacamole-client/pull/1028#discussion_r1835410771
########## guacamole-ext/src/main/java/org/apache/guacamole/net/auth/AbstractAuthenticatedUser.java: ########## @@ -29,13 +34,39 @@ public abstract class AbstractAuthenticatedUser extends AbstractIdentifiable implements AuthenticatedUser { + /** + * The logger for this class. + */ + private static final Logger LOGGER = LoggerFactory.getLogger(AbstractAuthenticatedUser.class); + + /** + * The server environment in which this Guacamole Client instance is + * running. + */ + private final Environment environment = LocalEnvironment.getInstance(); + // Prior functionality now resides within AbstractIdentifiable @Override public Set<String> getEffectiveUserGroups() { return Collections.<String>emptySet(); } + @Override + public boolean isCaseSensitive() { + try { + return environment.getCaseSensitivity().caseSensitiveUsernames(); + } + catch (GuacamoleException e) { + LOGGER.warn("Exception attempting to read the Guacamole configuration, " + + "usernames will be treated as case-sensitive.", e.getMessage()); Review Comment: Added via rebase. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@guacamole.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org