Projet

Général

Profil

Development #55383

erreurs ldap, tout le temps mentionner le serveur / bloc de config concerné

Ajouté par Frédéric Péters il y a presque 3 ans. Mis à jour il y a presque 3 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
05 juillet 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Je pense à ça sur les traces "connection timed out while retrieving group DNs" qui ne disent pas quel serveur LDAP est concerné mais à lire ldap_backend.py il y a d'autres erreurs où ça serait utile d'avoir une telle référence,

                log.error("account name authentication filter doesn't contain '%s'")
...
                                log.error('user_filter syntax error %r: %s', block['user_filter'], e)
...

ok il y en a pas mal où ça sera "le bloc que je suis occupé à configurer" mais quand même pas de mal à le préciser; et sur d'autres logs à proximité c'est déjà fait, ex:

                            log.debug(
                                '[%s] looking up dn for username %r using query %r', ldap_uri, username, query
                            )
...
                        log.error('[%s] user lookup failed, basedn %s not found', ldap_uri, user_basedn)

Fichiers

Révisions associées

Révision 31f25ccc (diff)
Ajouté par Valentin Deniaud il y a presque 3 ans

ldap: include server address in error logs (#55383)

Historique

#1

Mis à jour par Valentin Deniaud il y a presque 3 ans

  • Assigné à mis à Valentin Deniaud
#2

Mis à jour par Valentin Deniaud il y a presque 3 ans

#3

Mis à jour par Thomas Noël il y a presque 3 ans

faut mettre ldap_uri en premier dans « log.error('[%s] user bind failed: authz_id not found %r', ', '.join(authz_ids), ldap_uri) » (sinon ça semble ok)

#5

Mis à jour par Thomas Noël il y a presque 3 ans

  • Statut changé de Solution proposée à Solution validée
#6

Mis à jour par Valentin Deniaud il y a presque 3 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit 007de68bb985122ffe19c6f6ba70174a3da3e2b3
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Tue Jul 6 11:19:55 2021 +0200

    ldap: include server address in error logs (#55383)
#7

Mis à jour par Frédéric Péters il y a presque 3 ans

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

Formats disponibles : Atom PDF