mike-jumper commented on PR #1028: URL: https://github.com/apache/guacamole-client/pull/1028#issuecomment-2421872998
Hm ... maybe what I was thinking is actually backwards, and we should instead: * Use the `*-case-sensitive-usernames` properties to dictate only whether users authenticating _via that extension_ have case-sensitive usernames. * Extensions that consume usernames from other extensions _should_ honor the `isCaseSensitive()` value when comparing identities. * In the equality comparison for `AbstractIdentifiable`, we need some sane way to resolve what happens if you compare a case-sensitive identifier with a case-insensitive identifier. Otherwise things stop making sense. The JSON auth has a `json-case-sensitive-usernames` property but does not consume external identifiers, so the intent of that property _has_ to be declaring the intent of what other extensions should do with identifiers from JSON auth. For that to be useful, we would need to meaningfully consume `isCaseSensitive()`, but most of the logic around consuming external identifiers on the database side depends solely on the database's `*-case-sensitive-usernames` property. I'm starting to thing these changes have part of the right idea, and we need to change the rest to match. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
