Projet

Général

Profil

Bug #83425

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

Ajouté par Emmanuel Cazenave il y a 6 mois. Mis à jour il y a 4 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
13 novembre 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

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

Révision 24216c64 (diff)
Ajouté par Benjamin Dauvergne il y a 4 mois

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

Historique

#4

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

#5

Mis à jour par Benjamin Dauvergne il y a 5 mois

  • Assigné à mis à Benjamin Dauvergne
#6

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 :

#7

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 :

#8

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 :

#9

Mis à jour par Transition automatique il y a 4 mois

  • Statut changé de Résolu (à déployer) à Solution déployée
#10

Mis à jour par Transition automatique il y a environ 2 mois

Automatic expiration

Formats disponibles : Atom PDF