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