----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14148/ -----------------------------------------------------------
Review request for cloudstack and Abhinandan Prateek. Repository: cloudstack-git Description ------- DefaultUserAuthenticator maskes the _name varible in ComponentLifecycleBase making the setName() method not work as expected. This patch cleans up the code such that getName() will be getClass().getSimpleName() unless overridden in the Spring configuration. Diffs ----- plugins/user-authenticators/ldap/src/org/apache/cloudstack/ldap/LdapAuthenticator.java e62a3d8 plugins/user-authenticators/md5/src/com/cloud/server/auth/MD5UserAuthenticator.java e5b169f plugins/user-authenticators/plain-text/src/com/cloud/server/auth/PlainTextUserAuthenticator.java f102275 plugins/user-authenticators/sha256salted/src/com/cloud/server/auth/SHA256SaltedUserAuthenticator.java 91be922 server/src/com/cloud/server/auth/DefaultUserAuthenticator.java 952f724 Diff: https://reviews.apache.org/r/14148/diff/ Testing ------- Thanks, Darren Shepherd