Package: libnss-ldap
Version: 238-1
Severity: wishlist

Would it be possible to ask for the URI parameter instead of the host
in the debconf configuration? This would allow the user to specify not
only the hostname (or IP), but the port and "access method"
(ldap://<hostname>:<port>, ldaps:///, or ldapi:///) without having to
edit /etc/libnss-ldap.conf and remove it from debconf control.

I don't know how to make proper patches, but I can point to where the
relevant changes would have to be made. Sorry for the ugliness, I'm
going to go hit the "diff" manpage after I send this:

debian/config, line 54:
change from:  read_and_input('shared/ldapns/ldap-server', 'host', 'critical');
to: read_and_input('shared/ldapns/ldap-server', 'uri', 'critical');
because: more accurate name now

debian/libnss-ldap.postinst, line 38: 
change from: perl -i -p -e "s/$replacestring/$parameter $value/i
to: perl -i -p -e "s-$replacestring-$parameter $value-i
because: the URIs have slashes in them that confuse the regex, so
change the regex separator

debian/libnss-ldap.postinst, line 88: 
change from: change_value host "$RET"
to: change_value uri "$RET"
because: name changed in debconf db

debian/libnss-ldap.postinst, line 87: 
add: disable_param host
because: default config file has host uncommented, I don't think URI
and host get along together

debian/templates, line 80:
change from: 
Template: shared/ldapns/ldap-server
Type: string
_Default: 127.0.0.1
_Description: LDAP server host address
 Please enter the address of the LDAP server used.
 .
 Note: It is usually a good idea to use an IP address; this reduces risks
 of failure in the event name service is unavailable.
to: 
Template: shared/ldapns/ldap-server
Type: string
_Default: ldap://127.0.0.1/
_Description: LDAP server Uniform Resource Identifier
 Please enter the URI of the LDAP server used. This is a string in the
 form ldap://<hostname or IP>:<port>/ . ldaps:// or ldapi:// can also
 be used. The port number is optional.
 .
 Note: It is usually a good idea to use an IP address; this reduces risks
 of failure in the event name service is unavailable.


Again, sorry for not including proper patches to the files.

Reply via email to