Please review and test preliminary support for LDAP controls
Michael Ströder
michael at stroeder.com
Fri Mar 11 09:58:36 CET 2005
Deepak Giridharagopal wrote:
> On Tue, 2005-03-01 at 21:24 +0100, Michael Ströder wrote:
>
>>Anyway I'd appreciate every one's feedback whether the modifications in
>>class LDAPControl and the sub-module ldap.controls seems suitable. So
>>please bring your CVS working dir in sync and test. Thanks!
>
> It looks like ldapobject.py invokes the EncodeControlTuples function for
> some extended ldap operations, but not all of them. Ideally, every call
> to "_ldap_call" that takes controls as parameters should be converting
> LDAPControl arguments to tuples, right?
Right! Sorry, forgot to modify all the other LDAPObject methods as well.
> I've attached a patch.
Please commit it.
Ciao, Michael.
More information about the python-ldap
mailing list