On Tue, Feb 14, 2012 at 3:31 PM, Afkham Azeez <az...@wso2.com> wrote:
> > > On Tue, Feb 14, 2012 at 3:22 PM, Amila Jayasekara <ami...@wso2.com> wrote: > >> Hi All, >> >> We had a review discussion on Carbon Authenticators and following are >> the review notes. >> >> 1. There are some common logic that should be implemented in every >> authenticator. Currently invoking this logic is duplicated among >> authenticators. So to avoid that we are planning to come up with an >> abstract Authenticator implementation. >> 2. As per now AuthenticationHandler first invokes AuthenticationAdmin >> and then it calls chain of authenticators. The AuthenticationAdmin >> call is not necessary. We need to refactor code in such a way that >> authenticator it self will only handle authenticating logic (Rather >> than within the Handler). >> 3. Properly implement authenticator chaining pattern. Make use >> “isHandle” > > > Shouldn't this be isHandle*d*? > Yes.. we discussed today to change the name.. and suggestion was to rename it to canHandle().. Thanks & regards, -Prabath > > >> , “priority”, “isAuthenticated” methods and make only >> authenticator aware about the logic. >> 4. After cleaning up the API, implement “Basic Auth” authenticator. >> (As first step) >> >> Thanks >> AmilaJ >> >> -- >> Mobile : +94773330538 >> _______________________________________________ >> Carbon-dev mailing list >> Carbon-dev@wso2.org >> http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev >> > > > > -- > *Afkham Azeez* > Director of Architecture; WSO2, Inc.; http://wso2.com > Member; Apache Software Foundation; http://www.apache.org/ > * <http://www.apache.org/>** > email: **az...@wso2.com* <az...@wso2.com>* cell: +94 77 3320919 > blog: **http://blog.afkham.org* <http://blog.afkham.org>* > twitter: **http://twitter.com/afkham_azeez*<http://twitter.com/afkham_azeez> > * > linked-in: **http://lk.linkedin.com/in/afkhamazeez* > * > * > *Lean . Enterprise . Middleware* > > > _______________________________________________ > Carbon-dev mailing list > Carbon-dev@wso2.org > http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev > > -- Thanks & Regards, Prabath Mobile : +94 71 809 6732 http://blog.facilelogin.com http://RampartFAQ.com
_______________________________________________ Carbon-dev mailing list Carbon-dev@wso2.org http://mail.wso2.org/cgi-bin/mailman/listinfo/carbon-dev