[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-19-2008 14:51 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-19-08 14:51  
---------------------------------------------------------------------- 
Fine. I guess I can live without Asterisk, and definitely Asterisk can live
without me. (And about getting into legalities, I guess I've been there too
during this experiment, as someone with engineer's mind I did not even try
to interpret the Digium license, and no, I don't have an attorney to help
me, I live in a society where normal people still can cope with their lives
without having one). So from my part, you can close the bug reports. Though
I guess it's pretty easy to re-implement things based on the description,
in case someone has desire to do so. I'd be very happy with that.

Pitäkää tunkkinne. 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
03-19-08 14:51  tjormola       Note Added: 0084286                          
======================================================================