necouchman commented on PR #1028: URL: https://github.com/apache/guacamole-client/pull/1028#issuecomment-2450468883
> > ... Thoughts on expanding the scope from just `User` objects to include `UserGroup` objects in this? > > I agree. I'm sure this will be necessary and it makes sense to include this as part of the general configurable case sensitivity effort. Okay - I _think_ I've completed the re-work of the user code to only pull from the global configuration value, and removed all of the extension-based options. I have left the bits in place, mainly in the JDBC module, that pull from the `user.isCaseSensitive()` method, rather than pulling the environment directly - this tends to be a bit easier than injecting the environment everywhere it might be required, and also keeps the concern of whether or not a `User` identifier is case sensitive as part of the `User` object. I'll see if I can knock the `UserGroup` portions out real quick - shouldn't be too terribly difficult. And then...testing... -- 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
