Project

General

Profile

Bug #83425

Hôte manquant sur l'affichage de la commande ldap

Added by Emmanuel Cazenave 3 months ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Category:
-
Target version:
-
Start date:
13 November 2023
Due date:
% Done:

0%

Estimated time:
Patch proposed:
No
Planning:
No

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.

Associated revisions

Revision 24216c64 (diff)
Added by Benjamin Dauvergne about 1 month ago

ldap: set LDAP url on raised LDAPError for the technical information view (#83425)

History

#4

Updated by Thomas Noël (congés → 26 février) 3 months ago

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 :-)

#5

Updated by Benjamin Dauvergne 3 months ago

  • Assignee set to Benjamin Dauvergne
#6

Updated by Robot Gitea 3 months ago

  • Status changed from Nouveau to Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#7

Updated by Robot Gitea about 2 months ago

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

Nicolas Roche (nroche) a approuvé une pull request sur Gitea concernant cette demande :

#8

Updated by Robot Gitea about 1 month ago

  • Status changed from Solution validée to Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#9

Updated by Transition automatique about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF