mike-jumper commented on code in PR #1028: URL: https://github.com/apache/guacamole-client/pull/1028#discussion_r1828581893
########## guacamole-ext/src/main/java/org/apache/guacamole/net/auth/Identifiable.java: ########## Review Comment: I think we should probably remove the `isCaseSensitive()` function, at least from guacamole-ext, so that it does not potentially result in conflicting information between an implementation's `isCaseSensitive()` vs. the global configuration option. If it makes things easier to plumb within the JDBC auth, I don't see any issue with having a `isCaseSensitive()` function that's strictly internal to JDBC. -- 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
