allow ldap ipv6 uris

hello,

https://github.com/thexa4/fusiondirectory-plugins/commit/592ea777dd1cfda2d3789a0a35e21089369b3952

seems a nice fix to allow ldap ipv6 uris.

we could see to include it

Cheers

(from redmine: issue id 4484, created on 2016-01-27, closed on 2016-02-09)

• Changesets:
  • Revision 4a3eba60 by Côme Chilliet on 2016-02-01T10:34:58.000Z:

Fixes #4484 Allowing IPv6 in LDAP service URI

Conflicts:
	systems/admin/systems/services/ldap/class_serviceLDAP.inc

• Revision 3249897f by Côme Chilliet on 2016-02-01T10:35:31.000Z:

Fixes #4484 Allowing IPv6 in LDAP service URI

Conflicts:
	systems/admin/systems/services/ldap/class_serviceLDAP.inc

• Custom Fields:
  • Bug in version: 1.0.9.3

Pushed

(from redmine: written on 2016-01-28)

I think it not work.

I have this error:

The field 'LDAP URI' contains invalid characters. 'ldap://fe80::a00:27ff:fe8e:ae2e:389/' are not allowed!

"ldap://fe80::a00:27ff:fe8e:ae2e:389/"

Example: ldap://fill-in-your-servers-dns-name:389/

(from redmine: written on 2016-01-28)

hello,

yes format look like ldap://[3ffe:2000:0:1:e0be:1872:d4f8:6b2c] and its not working either.

i have reverted it to get 1.0.9.3 and move this to 1.0.10

Cheers

(from redmine: written on 2016-01-28)

The format the regexp expect is this: «ldap://[3ffe:2000:0:1:e0be:1872:d4f8:6b2c]/» So, [] around the IPv6 and a final /. This final slash is also needed for IPv4 and has always been. I can make it optional if you think it’s better (We must not lose too much time on this, this ldap service is not really usefull nor critical from what I know, maybe the best is just to revert the revert and make the slash optional. It won’t hurt anyway.)

(from redmine: written on 2016-02-01)

hello,

i agree remove the /

Cheers

(from redmine: written on 2016-02-01)

Close issue

(from redmine: written on 2016-02-09)

closed

added Added label