Page MenuHomekolab.org

Fix issue with re-binding to an LDAP connection (T1318)
ClosedPublic

Authored by machniak on Jun 21 2016, 11:26 AM.

Details

Summary

Fixes T1318

Note that set_entry_attributes() will now use two LDAP connections, which may
or may not be what we want. I didn't find a simple way of using ldap_priv connection
in entry_dn() and get_entry_attributes().

Diff Detail

Repository
rP pykolab
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

machniak updated this revision to Diff 411.Jun 21 2016, 11:26 AM
machniak retitled this revision from to Fix issue with re-binding to an LDAP connection (T1318).
machniak updated this object.
machniak edited the test plan for this revision. (Show Details)
machniak added a reviewer: PyKolab Developers.
machniak updated this revision to Diff 412.Jun 22 2016, 11:43 AM

Get rid of unbind() and close ldap_priv connection on disconnect()

machniak updated this revision to Diff 413.Jun 22 2016, 12:05 PM

Call self._bind() before using self.entry_dn() method. @FIXME: _bind() should
probably be called explicitely by entry_dn()

vanmeeuwen accepted this revision.Jun 29 2016, 11:21 AM
vanmeeuwen added a reviewer: vanmeeuwen.
This revision is now accepted and ready to land.Jun 29 2016, 11:21 AM
This revision was automatically updated to reflect the committed changes.