[asterisk-bugs] [Asterisk 0012235]: [patch] Improve LDAP server configuration for res_config_ldap

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12235 
====================================================================== 
Reported By:                tjormola
Assigned To:                Corydon76
====================================================================== 
Project:                    Asterisk
Issue ID:                   12235
Category:                   Resources/res_config_ldap
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     needs license
Asterisk Version:           1.6.0-beta5 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             03-17-2008 11:35 CDT
Last Modified:              03-18-2008 18:57 CDT
====================================================================== 
Summary:                    [patch] Improve LDAP server configuration for
res_config_ldap
Description: 
This patch alters res_config_ldap.c so that full LDAP URL is used
internally to represent the LDAP server to use instead of using hostname
and port variables. It's also possible to specify the full URL in
res_ldap.conf using the new url keyword instead of just specifying host and
port (but this works too, of course, so existing configurations should not
break due to this change). Rationale behind this is that using a URL you
can specify extra settings for the connections that are not available
otherwise (e.g. you can use ldapi:// urls when connecting to a local
OpenLDAP server, or you can specify the scope of the queries, see
ldap_url(3) manual page for more info).

Also currently the port setting code is broken. It generates invalid LDAP
URLs if both host and port of the LDAP server is specified in the
configuration file.

The patch also fixes setting of the protocol. You can specify this in the
configuration file, but the setting was never applied to the LDAP
connection. Thus all connections were using LDAPv2 by default (at least
when Asterisk is compiled against OpenLDAP 2.4).
====================================================================== 

---------------------------------------------------------------------- 
 tjormola - 03-18-08 18:57  
---------------------------------------------------------------------- 
I feel a bit awkward giving out my personal contact information for such
small things like these patches in this contributor license agreement
thingy. Can't I just declare that this patch and the other I've recently
submitted are public domain. Everyone, including Digium Inc., is free to do
what ever they please with it, including re-licensing under any license. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-18-08 18:57  tjormola       Note Added: 0084213                          
======================================================================