Development #51353
différencier la raison "ldap is down"
0%
Description
except (ldap.TIMEOUT, ldap.CONNECT_ERROR, ldap.SERVER_DOWN):
return False, u'ldap is down'
et c'est "ldap is down" qui se trouve dans les logs pour les trois exceptions, ça peut parfois être utile pour analyser de savoir laquelle exception des trois a joué.
Fichiers
Révisions associées
Historique
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0001-ldap-differentiate-errors-during-bind-51353.patch 0001-ldap-differentiate-errors-during-bind-51353.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Valentin Deniaud il y a environ 3 ans
Mis à jour par Thomas Noël il y a environ 3 ans
"did you forget to declare the TLS certificate in /etc/..." je suis pas fan de proposer une telle fausse piste (j'ai vu qu'elle est ailleurs dans le code mais c'est pas une raison ;) ).
Je ne pense pas utile d'afficher le block['url'], elle est déjà affichée lors de l'analyse du résultat du bind dans get_connections : « 'admin bind failed on %s: %s', url, error ».
Bref, rester sur des petits messages tout simples, "timeout", "ldap is down", "connection error".
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0001-ldap-differentiate-errors-during-bind-51353.patch 0001-ldap-differentiate-errors-during-bind-51353.patch ajouté
Okay
Mis à jour par Thomas Noël il y a environ 3 ans
Je pense qu'on peut quand même utiliser des return :)
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Fichier 0001-ldap-differentiate-errors-during-bind-51353.patch 0001-ldap-differentiate-errors-during-bind-51353.patch ajouté
Ce patch est trop compliqué, je m'y perds !
Mis à jour par Thomas Noël il y a environ 3 ans
- Statut changé de Solution proposée à Solution validée
Mis à jour par Valentin Deniaud il y a environ 3 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit c823bb76a6389dc294f63d263e2999057bcec163
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date: Wed Feb 24 14:42:29 2021 +0100
ldap: differentiate errors during bind (#51353)Mis à jour par Frédéric Péters il y a environ 3 ans
- Statut changé de Résolu (à déployer) à Solution déployée
ldap: differentiate errors during bind (#51353)