LDAP::convert and LDAP::fix functions are causing problems

It seems FD historically dealt with «sanitarized» versions of dn without the characters «,()/"» These are removed by LDAP::convert and added back by LDAP::fix. It’s causing problems such as for #4456 last fix.

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

• Relations:
  • copied_to #4495
  • copied_to #5322
• Changesets:
  • Revision c2f8ce2d by Côme Chilliet on 2016-01-26T09:45:35.000Z:

Fixes #4480 Disabling convert and fix to see if it breaks anything

• Revision cddb8001 by Côme Chilliet on 2016-01-26T09:46:52.000Z:

Fixes #4480 Disabling convert and fix to see if it breaks anything

• Revision 1f42fc5f by Côme Chilliet on 2016-01-26T09:47:47.000Z:

Fixes #4480 Disabling convert and fix to see if it breaks anything

• Revision be68e898 by Côme Chilliet on 2016-01-27T03:17:42.000Z:

Fixes #4480 Added ldap_escape and start using it in appropriate places

• Revision d9ad341b by Côme Chilliet on 2016-01-27T03:18:29.000Z:

Fixes #4480 Added ldap_escape and start using it in appropriate places

• Revision ca3e051d by Côme Chilliet on 2016-01-27T03:19:42.000Z:

Fixes #4480 Added ldap_escape and start using it in appropriate places

• Revision 19594cb8 by Côme Chilliet on 2016-01-27T03:20:12.000Z:

Fixes #4480 Putting SPECIALS_OVERRIDE back to TRUE for 1.0.9.3

• Revision af690c29 by Côme Chilliet on 2016-01-28T09:38:47.000Z:

Fixes #4480 Removing useless compute_dn and adding ldap_escape to the others

• Revision a07ab4b6 by Côme Chilliet on 2016-01-28T09:40:07.000Z:

Fixes #4480 Removing useless compute_dn and adding ldap_escape to the others

• Revision 590e1725 by Côme Chilliet on 2016-02-08T04:51:45.000Z:

Fixes #4480 Fixed default implementation of compute_dn

• Revision 92ddbb4e by Côme Chilliet on 2016-02-08T04:52:22.000Z:

Fixes #4480 Fixed default implementation of compute_dn

• Custom Fields:
  • Bug in version: 1.0.9.2

hello,

yes as discussed on irc they should be rerwitten/removed and a function like https://secure.php.net/manual/en/function.ldap-escape.php should be written to be used in place of fix and convert

Cheers

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

By bmortier on 2017-09-02T15:26:10 (imported from GitLab)

Fixed #4456 problem with a work-around. We should generalize the use of ldap_escape and fix dn handling for 1.0.10

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

By Côme Chilliet on 2017-09-02T15:26:10 (imported from GitLab)

Close issue

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

By Jonathan Swaelens on 2017-09-02T15:26:10 (imported from GitLab)

closed

By Jonathan Swaelens on 2017-09-02T15:26:10 (imported from GitLab)

added Changed label

By bmortier on 2018-10-05T11:27:06 (imported from GitLab)