Ben Alex wrote:
Hi Scott

I am inclined to leave it as-is, as detecting the type returned by getPrincipal() is rather easy to do, and can be encapsulated into a static utility method in any event. Also, as we're in the 1.0.0 RC phase, changing a relatively central interface like Authentication should be viewed with caution.
Sounds reasonable. My only concern had been that third party applications using Acegi expected certain return value and when you swapped providers there was no guarantee of consistency (and sometimes no way to update the code). I actually experienced this problem with QuickBuild and I had to provide a modified Acegi jar in order for the application to work properly. Maybe this just means that applications should be calling authentication.getUserDetails instead of getPrincipal.

-Scott


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642
_______________________________________________
Home: http://acegisecurity.org
Acegisecurity-developer mailing list
Acegisecurity-developer@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/acegisecurity-developer

Reply via email to