[Freeipa-devel] [PATCH 64] Implement password based session login

John Dennis jdennis at redhat.com
Mon Feb 27 16:43:38 UTC 2012


Attached is a revised patch, it addresses the following concerns raised 
during review:

* The version in ipa.conf has been bumped.

* Rob reported duplicate session cookies being returned. As far as I can 
tell this was due to a Python bug where it reused the value of a default 
keyword parameter from a previous invocation rather than re-initializing 
it. Workaround is to change the default value from [] to the value to 
None and create an empty list if the arg is None.

* Rob reported two test failures, one for ERRNO (e.g. **1234**) not 
being present in the docstring for an error I added and the other was 
for a change in the wsgi dispatch route() method that showed up in 
test_rpcserver.py



-- 
John Dennis <jdennis at redhat.com>

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-jdennis-0064-1-Implement-password-based-session-login.patch
Type: text/x-patch
Size: 26124 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20120227/2909458b/attachment.bin>


More information about the Freeipa-devel mailing list