Re: Pondering a user login validation bug.

Ramsey Gurley <[email protected]>
Newsgroups gmane.comp.web.webobjects.wonder-disc
Message-ID <[email protected]>
Oops :-) Slight oversight there.

It looks like the ERTwoFactorAuthenticationConfig needs an abstract method to check to see if a user should be allowed to login before attempting to verify password.  Thanks for pointing it out.  I'll fix it tonight.

Ramsey

On Dec 5, 2012, at 9:43 AM, James Cicenia wrote:

> Scenario -
> 
> User is in PRE_ACTIVATION status
> 
> In the login direct action you have:
> 
> 
> 
> if(eo instanceof ERAuthenticationRequest) {
> 	ERAuthenticationRequest authRequest = (ERAuthenticationRequest)eo;
> 	authRequest.takeValuesFromRequest(request);
> 	try {
> 		eo.validateForSave();
> 	} catch(ValidationException e) {
> 		WOComponent fail = authRequest.clazz().authenticationConfig().loginFailurePage();
> 
> 
> However, it won't throw an exception because the eo is an ERTwoFactorAuthenticationRequest. Its validateForSave is nothing special.
> In your ERUser class you do have an appropriate looking validateForSave.
> 
> Thoughts?
> 
> James
> 
> ------------------------------------------------------------------------------
> LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
> Remotely access PCs and mobile devices and provide instant support
> Improve your efficiency, and focus on delivering more value-add services
> Discover what IT Professionals Know. Rescue delivers
> http://p.sf.net/sfu/logmein_12329d2d_______________________________________________
> Wonder-disc mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/wonder-disc


------------------------------------------------------------------------------
LogMeIn Rescue: Anywhere, Anytime Remote support for IT. Free Trial
Remotely access PCs and mobile devices and provide instant support
Improve your efficiency, and focus on delivering more value-add services
Discover what IT Professionals Know. Rescue delivers
http://p.sf.net/sfu/logmein_12329d2d
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.