Bug #83425
Hôte manquant sur l'affichage de la commande ldap
0%
Description
Une erreur 'invalid credentials' sur un raccordement ldap, sur /manage/tech-info/ il manque l'hôte dans la commande ldap : ldapsearch -v -H -D ....
.
Alors que l'hôte est bien dans la conf.
Révisions associées
Historique
Mis à jour par Thomas Noël il y a 6 mois
Très certainement parce qu'on n'envoie pas config['ldap_uri']
en cas d'erreur, cf tech_info dans src/authentic2/manager/views.py :
try: conn = backend.get_connection(config, raises=True) except ldap.LDAPError as e: config['error'] = True config['errmsg'] = str(e) else: # retrieve ldap uri, not directly visible in configuration block config['ldap_uri'] = conn.get_option(ldap.OPT_URI)
mais comme la connexion n'a pas réussie, on n'a pas de conn.get_option(...) possible. Et il semble bien que ldap.LDAPError ne contienne pas l'info non plus (https://www.python-ldap.org/en/python-ldap-3.3.0/reference/ldap.html#ldap.LDAPError)
Ça semble mal parti... on pourrait au pire poser un affichage de la liste des config['url'] (on dirait que c'est toujours une liste, vu le code src/authentic2/backends/ldap_backend.py qui gère les "_TO_ITERABLE"... ok j'arrête de creuser ici :-)
Mis à jour par Robot Gitea il y a 5 mois
- Statut changé de Nouveau à Solution proposée
Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/200
- Titre : ldap: set LDAP url on raised LDAPError for the technical information view (#83425)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/200/files
Mis à jour par Robot Gitea il y a 4 mois
- Statut changé de Solution proposée à Solution validée
Nicolas Roche (nroche) a approuvé une pull request sur Gitea concernant cette demande :
Mis à jour par Robot Gitea il y a 4 mois
- Statut changé de Solution validée à Résolu (à déployer)
Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/200
- Titre : ldap: set LDAP url on raised LDAPError for the technical information view (#83425)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/200/files
Mis à jour par Transition automatique il y a 4 mois
- Statut changé de Résolu (à déployer) à Solution déployée
ldap: set LDAP url on raised LDAPError for the technical information view (#83425)