Project

General

Profile

Development #51353

différencier la raison "ldap is down"

Added by Frédéric Péters 2 days ago. Updated about 17 hours ago.

Status:
Solution validée
Priority:
Normal
Category:
-
Target version:
-
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

#2

Updated by Valentin Deniaud about 18 hours ago

  • Assignee set to Valentin Deniaud
#3

Updated by Valentin Deniaud about 18 hours ago

#5

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".

#7

Updated by Thomas Noël about 17 hours ago

Je pense qu'on peut quand même utiliser des return :)

#9

Updated by Thomas Noël about 17 hours ago

  • Status changed from Solution proposée to Solution validée

Also available in: Atom PDF