Development #51353
différencier la raison "ldap is down"
Start date:
23 Feb 2021
Due date:
% Done:
0%
Estimated time:
Patch proposed:
Yes
Planning:
No
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é.
Files
History
Updated by Valentin Deniaud about 18 hours ago
- Patch proposed changed from No to Yes
- Status changed from Nouveau to Solution proposée
- File 0001-ldap-differentiate-errors-during-bind-51353.patch 0001-ldap-differentiate-errors-during-bind-51353.patch added
Updated by Thomas Noël about 17 hours ago
"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".
Updated by Valentin Deniaud about 17 hours ago
- File 0001-ldap-differentiate-errors-during-bind-51353.patch 0001-ldap-differentiate-errors-during-bind-51353.patch added
Ce patch est trop compliqué, je m'y perds !
Updated by Thomas Noël about 17 hours ago
- Status changed from Solution proposée to Solution validée