ldapi:// with path

[Arran Cudbard-Bell]

> On 30 Jun 2015, at 19:12, Alan DeKok <aland at deployingradius.com> wrote:
> 
> On Jun 30, 2015, at 5:55 PM, Michael Ströder <michael at stroeder.com> wrote:
>> 
>> Re-tested with 8655446068a7bbec8a6ecc148418926fea9fc2d0:
>> 
>> 	server = 'ldap://localhost'
>> 	port = 1390
>> 
>> now results in ldap://localhost:389
> 
>  I've pushed a fix.

You pushed (helpful) changes for the case where we don't have ldap_initialize.

We know on OPs system he has ldap_initialize because the code is producing URLs, so it'll still be broken in the OPs case.

This says when a port does not exist in the string, don't set it.

http://www.openldap.org/devel//gitweb.cgi?p=openldap.git;a=blob;f=libraries/libldap/url.c;h=dd1fa21f6363a681b418a5e0ff0797d8fa845b4a;hb=HEAD#l943

Unless LDAP_PVT_URL_PARSE_DEF_PORT was passed in.

So I have no idea why it's still messed up. I guess i'll run it under lldb and find exactly where libldap is setting the port.

-Arran

Arran Cudbard-Bell <a.cudbardb at freeradius.org>
FreeRADIUS development team

FD31 3077 42EC 7FCD 32FE 5EE2 56CF 27F9 30A8 CAA2

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 872 bytes
Desc: Message signed with OpenPGP using GPGMail
URL: <http://lists.freeradius.org/pipermail/freeradius-users/attachments/20150630/af229368/attachment.sig>