Discussion :

[CinePhil]

Bonjour,

J'essaie un ldap_bind pour vérifier un mot de passe (l'existence de l'utilisateur est préalablement vérifiée dans le programme, ainsi que son statut d'utilisateur LDAP ou externe) :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
			/** 3) Vérifier le mot de passe de l'utilisateur */
			if($utilisateur->getUtiLdap() == true)
			{
				// Vérification via LDAP
				try 
				{
					$ds = ldap_connect(Configuration::get('ldap_host'));
					ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3);
					$util = ldap_bind($ds, 'uid='.$username.','.Configuration::get('ldap_base_dn'), $password);
					ldap_close($ds);
				}
				catch (Exception $e)
				{
					// Mauvais mot de passe par rapport au LDAP
					$this->setObjErreur();
					$this->getObjErreur()->ajouterErreur('Mauvais mot de passe (LDAP) !'); // TODO i18n
 
					// Réaffichage du formulaire de connexion
					$this->afficherPageConnexion($this->getObjErreur()->getListeErreurs());
				}
			} // Fin de if($utilisateur->getUtiLdap() == true)

Malgré le try / catch, comme j'ai quand même le warning suivant qui s'affiche en dev (en prod, je n'afficherai plus les erreurs) :

Warning: ldap_bind(): Unable to bind to server: Invalid credentials in mon/chemin/vers/mon/programme.php on line 68

Pire : Le code à l'intérieur du catch ne s'exécute pas et j'ai une page blanche.

La ligne 68 est bien entendu la ligne du ldap_bind.

Et si je saisis le bon mot de passe, pas d'erreur donc la connexion au ldap n'est pas en cause.

Ce comportement est-il normal ?

[Celira]

Pour l'exception (ou plutôt l'absence d'exception), je dirais que oui, c'est normal : rien dans la doc ne me laisse à penser que les fonctions ldap lancent des exceptions nativement (peut-être en PHP7, mais tu as taggué ton thread PHP 5.6) Donc, à moins que tu n'ais intégré une gestion d'erreurs personnalisée, l'absence d'exception ne m'étonne pas.

Pour gérer l'erreur, je crains qu'il ne faille faire du code moche pour générer une exception manuellement Tu peux mettre quelque chose de ce genre en place :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
$util = @ldap_bind($ds, 'uid='.$username.','.Configuration::get('ldap_base_dn'), $password);
if ($util === false) {
   // Oops...
    throw new Exception("LDAP bind dans les choux !");
}

Eventuellement, tu peux enrichir ton exception en te basant sur ce commentaire de la doc : http://php.net/manual/fr/function.ldap-bind.php#103034 et des fonctions comme ldap_​error et ldap_​errno

[CinePhil]

C'est ce que j'avais fait au départ et je voulais faire ça plus propre mais tant pis.

Comme je vais désactiver l'affichage des erreurs en prod (ou au moins des warnings), ça devrait suffire.
Je reviens donc à mon code de départ.

Merci.