DefaultUserAuthenticator masks the _name variable in ComponentLifecycleBase effectively making it so that no UserAuthenticator gets the default name of getClass().getSimpleName().

Then each authenticator, except LDAP, does something like

if (name == null) {
  name = "MD5";
}

So instead of the default MD5UserAuthenticator it comes up with MD5 as its name.

So my problem is that LDAP currently has a null name on getName(). So which should I do? Add

if (name == null) {
  name = "LDAP";
}

Or remove the masked variable in DefaultUserAuthenticator such that all authenticators names will become the classname. I just need it to be consistent for what I'm doing at the moment. I'd personally prefer that they all just get the default name of getSimpleName(). But I don't know if there was some reason it was done this way.

Darren

Reply via email to