DaanHoogland commented on PR #8370: URL: https://github.com/apache/cloudstack/pull/8370#issuecomment-1860380779
is it wise to allow for empty passwords? I may be misinterpreting this code, but I would not want this in a default install. Or is this only applicable for LDAP logins? BTW, the update is not necessary if the workaround is mentioned in the release notes. To my knowledge this fix is directed only at installations using LDAP, as mentioned in https://lists.apache.org/thread/tqtcwsfb0knmvqct1hlow4ty2nc2w2j2, is it? these are questions/discussion points, no comments on the code. -- 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]
