[Checkins] SVN: PluggableAuthService/trunk/plugins/CookieAuthHelper.py Only use the __ac_name field if __ac_password is also present. This fixes a

Wichert Akkerman wichert at wiggy.net
Tue Jan 23 04:26:19 EST 2007


Oops, sorry about that. Seems I haven't tried vim on my macbook properly
yet.

Wichert.

Previously Jens Vagelpohl wrote:
> Wichert, you just introduced a tab on the line starting with "if  
> login...". Please fix.
> 
> jens
> 
> 
> On 23 Jan 2007, at 10:14, Wichert Akkerman wrote:
> 
> >Log message for revision 72192:
> >  Only use the __ac_name field if __ac_password is also present.  
> >This fixes a
> >  login problem for CMF sites where the login name was remembered  
> >between
> >  sessions with an __ac_name cookie.
> >
> >
> >Changed:
> >  U   PluggableAuthService/trunk/plugins/CookieAuthHelper.py
> >
> >-=-
> >Modified: PluggableAuthService/trunk/plugins/CookieAuthHelper.py
> >===================================================================
> >--- PluggableAuthService/trunk/plugins/CookieAuthHelper.py	 
> >2007-01-22 22:35:05 UTC (rev 72191)
> >+++ PluggableAuthService/trunk/plugins/CookieAuthHelper.py	 
> >2007-01-23 09:14:45 UTC (rev 72192)
> >@@ -113,7 +113,7 @@
> >         # Look in the request.form for the names coming from the  
> >login form
> >         login = request.form.get('__ac_name', '')
> >
> >-        if login:
> >+	if login and request.form.has_key('__ac_password'):
> >             creds['login'] = login
> >             creds['password'] = request.form.get('__ac_password', '')
> >
> >
> >_______________________________________________
> >Checkins mailing list
> >Checkins at zope.org
> >http://mail.zope.org/mailman/listinfo/checkins
> 

-- 
Wichert Akkerman <wichert at wiggy.net>    It is simple to make things.
http://www.wiggy.net/                   It is hard to make things simple.


More information about the Checkins mailing list