Jasig/cas — pull request #294 by laszlovandenhoek (2013-07-26 14:47:56+00:00)
Title: Don't hard-code the name of a Logger to be the name of a particular subclass

The Logger of AbstractAuthenticationManager is now named for one of its concrete children, AuthenticationManagerImpl. This is confusing (although not necessarily incorrect, but that's a discussion about semantics), because log statements actually originating from AAM are rendered as coming from AMI.

This becomes more than a nuisance when you subclass AbstractAuthenticationManager. In this scenario, AuthenticationManagerImpl is never involved at any point, but still the Logger is named after it, so you're sent a wild goose chase trying to find the responsible code.

This change fixes the problem for classes extending AbstractAuthenticationManager, while preserving the old behaviour for AuthenticationManagerImpl.