From: <mi...@st...> - 2007-03-20 20:55:50
|
HI! I'd like to release python-ldap 2.3 within the next weeks. For this release I'd like to get rid of using deprecated OpenLDAP C SDK functions. Since my C skills are limited I'd appreciate if somebody could take over implementing a new C function l_ldap_str2dn(). I've collected my thoughts in the SF tracker item SF#1657848 (see current content below). Ciao, Michael. -- Michael Ströder E-Mail: mi...@st... http://www.stroeder.com -------- Original Message -------- Subject: [ python-ldap-Bugs-1657848 ] ldap_value_free() deprecated Date: Sun, 18 Mar 2007 04:49:49 -0700 From: SourceForge.net <no...@so...> To: no...@so... Bugs item #1657848, was opened at 2007-02-12 11:16 Message generated for change (Comment added) made by stroeder You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=102072&aid=1657848&group_id=2072 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: None Group: None Status: Open Resolution: None Priority: 8 Private: No Submitted By: Michael Ströder (stroeder) Assigned to: Nobody/Anonymous (nobody) Summary: ldap_value_free() deprecated Initial Comment: Current ldap.h lists ldap_value_free() as deprecated. ldap_values_free_len() shall be used instead. ---------------------------------------------------------------------- >Comment By: Michael Ströder (stroeder) Date: 2007-03-18 12:49 Message: Logged In: YES user_id=64920 Originator: YES Hmm, since Python strings are similar to struct berval probably OpenLDAP's function ldap_bv2dn() should be used instead ldap_str2dn() within l_ldap_str2dn(). ---------------------------------------------------------------------- Comment By: Michael Ströder (stroeder) Date: 2007-03-18 11:52 Message: Logged In: YES user_id=64920 Originator: YES It seems ldap_value_free() is solely used in l_ldap_explode_dn() and ldap_explode_rdn(). These functions in turn use OpenLDAP's functions ldap_explode_dn() and ldap_explode_rdn() which are also marked as deprecated. So the best thing is for upcoming python-ldap 2.3 1. to add a new function l_ldap_str2dn() to functions.c, 2. add a new Python wrapper function str2dn() to Lib/ldap/functions.py, 3. to remove l_ldap_explode_dn() and ldap_explode_rdn() from functions.c and 4. modify pure Python wrapper function explode_dn() and explode_rdn() in ldap.functions to use the new Python function ldap.functions.str2dn() for preserving backward compability. This might change the behaviour of ldap.explode_dn() to return hex-escaped chars for NON-ASCII chars. This is odd anyway so personally I don't care. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=102072&aid=1657848&group_id=2072 |