Upon looking at this further, I don't feel that the Authenticator interface should be changed - supporting listeners is an additional feature IMO. That is, just because an Authenticator exists, it doesn't necessarily mean that listeners can be supported (depending on the environment).
I believe we instead need to make an AuthenticationListenerRegistrar interface and that should have the add/remove methods and the default Authenticator implementation(s) should implement the AuthenticationListenerRegistrar interface. This achieves the same result and is a little 'cleaner' IMO. Any objections? Les On Tue, Jan 11, 2011 at 11:23 PM, Kalle Korhonen (JIRA) <[email protected]> wrote: > Add addAuthenticationListener() and removeAuthenticationListener() operations > to Authenticator interface > -------------------------------------------------------------------------------------------------------- > > Key: SHIRO-233 > URL: https://issues.apache.org/jira/browse/SHIRO-233
