Discussion :

[CinePhil]

Bonjour,

Voici le morceau de mon code qui vérifie le login d'un utilisateur via LDAP :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
	// Vérification de la présence de l'utilisateur dans le LDAP
	try
	{
		$ds = ldap_connect(LDAP_HOST);
		ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3);
		$util = ldap_bind($ds, 'uid='.$user.','.UTIL_BASE_DN, $password);
		ldap_close($ds);
	}
	catch (Exception $e)
	{
		$error_msg = 'Erreur LDAP : '.$e;
		require RACINE.'public/page.phtml';
		exit;
	}

Si je tape un utilisateur et un mot de passe valide, ça fonctionne.

Si je tape un utilisateur valide mais un mauvais mot de passe, au lieu que l'erreur LDAP soit récupérée par le catch et traitée en gestion d'erreur, j'ai directement ceci à l'affichage :

Warning: ldap_bind(): Unable to bind to server: Invalid credentials in [chemin vers mon programme] on line [numéro de la ligne $util = ldap_bind($ds, 'uid='.$user.','.UTIL_BASE_DN, $password);]

Est-ce parce que l'erreur est un Warning ?
Si oui, comment faire pour que le catch fonctionne ?

[Spartacusply]

C'est ça, un warning n'est pas une exception.

Tu peux ne pas afficher le warning et tester le résultat du ldap_bind :

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
// Vérification de la présence de l'utilisateur dans le LDAP
	try
	{
		$ds = ldap_connect(LDAP_HOST);
		ldap_set_option($ds, LDAP_OPT_PROTOCOL_VERSION, 3);
		$util = @ldap_bind($ds, 'uid='.$user.','.UTIL_BASE_DN, $password);
                ldap_close($ds);
		if ($util) {
                    //Success
                } else {
                    //Fail
                }
	}
	catch (Exception $e)
	{
		$error_msg = 'Erreur LDAP : '.$e;
		require RACINE.'public/page.phtml';
		exit;
	}

[CinePhil]

OK Merci.
J'ai changé mon code.

Il y a toujours le Warning, le cas échéant, bien sûr.
Selon le résultat de mes tests, je verrai plus tard s'il faut que je capture le libellé du Warning.

Pour le moment, on va dire que c'est résolu.