jmuehlner commented on PR #1028: URL: https://github.com/apache/guacamole-client/pull/1028#issuecomment-2415191432
> @mike-jumper You might take a look and make sure that what I did is sane - I couldn't pull the `JDBCEnvironment` injection into the `HistoryTrackingConnection` class, so I ended up using the `isCaseSensitive()` method from the `currentUser`. I _think_ this is sane, but let me know if anything seems off about that... It looks like this should work, but it's a little goofy to be reaching through the user to get access to the environment. Consider just injecting the environment into the `UserContext`, which you can do by using a factory when it's constructed. -- 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
