Jasig/cas — pull request #169 by serac (2012-11-15 14:45:50+00:00)
Title: CAS-1181 Logging Tweaks in AbstractAuthenticationManager

From CAS-1181:
Further testing reveals AuthenticationHandler changes are not sufficient to remedy the issue. Code review and testing indicates that logging changes in AbstractAuthenticationManager introduced in the LPPE changesets produce stack traces under nominal circumstances with all authentication handlers.

This pull provides special treatment of AuthenticationException in logAuthenticationHandlerError, without which a stack trace is produced for all AuthenticationHandler errors. The change here logs AuthenticationExceptions, which are thrown for non-error conditions such as authn failures, at INFO level without a stack trace. Some additional log cleanup is included while we're making logging changes in this component.