[authentification] explode uniquement le premier ":"

[wouldsmina]

Si le caractère ":" est contenu dans le mot de passe, explode() l'interprétait comme un séparateur de champ. Ce patch limite simplement explode() à 2 champs.

[prytoegrian]

Mais je connais ça, non ?
Question en bonus : Et si c'est l'autre partie qui a un : ?

[wouldsmina]

Mais je connais ça, non ?

Je pensais aussi que c'était corrigé!

Question en bonus : Et si c'est l'autre partie qui a un : ?

l'autre partie == login. On autorise pas la création d'un login avec des caractères spéciaux... sauf avec ldap

[prytoegrian]

C'est problématique, ça veut dire que ces profils là ne pourront jamais se connecter. Je sais pas ce qui est considéré comme une bonne pratique en nommage LDAP, mais ça mériterait au moins de communiquer pour éviter les mauvaises surprises. Je propose un truc pérenne plutôt qu'une com volatile

[wouldsmina]

Atta, j'ai pas d'openldap de test sous la main, mais, déja, pour l'AD il n'est pas possible de mettre ":" dans le login.

[prytoegrian]

Viens avec moi du côté obscur : https://github.com/libertempo/docker

[wouldsmina]

c'est confirmé, pas possible d'ajouter un user avec : dedans

[prytoegrian]

OK, je m'y colle dès qu'elle est prête pour la QA.

[prytoegrian]

BPM !