Project

General

Profile

Development #61128

ne pas alerter à chaque timeout LDAP "connection timed out while retrieving group DNs"

Added by Frédéric Péters 4 months ago. Updated 4 months ago.

Status:
Fermé
Priority:
Normal
Category:
-
Target version:
-
Start date:
26 Jan 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Pour tous les utilisateurs il y a populate_user_groups et appel à get_ldap_group_dns et si ça timeout là-dedans,

            except ldap.TIMEOUT:
                log.error('[%s] connection timed out while retrieving group DNs', ldap_uri)

et le résultat ça peut être beaucoup d'erreurs. On pourrait pour un "bloc" LDAP notifier uniquement la première fois ?


Files

Associated revisions

Revision d65690e3 (diff)
Added by Benjamin Dauvergne 4 months ago

ldap: factorize get_users per block actions (#61128)

Revision ebab6fbf (diff)
Added by Benjamin Dauvergne 4 months ago

ldap: do not continue on timeout (#61128)

Revision 1a446f7c (diff)
Added by Benjamin Dauvergne 4 months ago

ldap: always report count of synchronized users (#61128)

History

#1

Updated by Benjamin Dauvergne 4 months ago

  • Assignee set to Benjamin Dauvergne
#4

Updated by Paul Marillonnet 4 months ago

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

Ok.

#5

Updated by Paul Marillonnet 4 months ago

Paul Marillonnet a écrit :

Ok.

(Et, pour être précis, je note bien que 0002 va plus loin que ce que la description du ticket voudrait, c’est ok pour moi.)

#6

Updated by Benjamin Dauvergne 4 months ago

Paul Marillonnet a écrit :

Paul Marillonnet a écrit :

Ok.

(Et, pour être précis, je note bien que 0002 va plus loin que ce que la description du ticket voudrait, c’est ok pour moi.)

À mon avis dès le premier timeout ensuite la connexion est cassée et va faire timeout tout le temps (enfin ça semble être le comportement vu), c'est pour cela que j'ai fait 0002 tel qu'il est.

#7

Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Solution validée to Résolu (à déployer)
commit 1a446f7c756d5c0cc9e952746f41f2af60723081
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Jan 27 12:20:22 2022 +0100

    ldap: always report count of synchronized users (#61128)

commit ebab6fbf4cfe68fd0351281b42ded8353845158a
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Jan 27 12:16:46 2022 +0100

    ldap: do not continue on timeout (#61128)

commit d65690e3e0785cf84ac2936cbedc42f71c124705
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Jan 27 11:35:19 2022 +0100

    ldap: factorize get_users per block actions (#61128)
#8

Updated by Transition automatique 4 months ago

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

Updated by Transition automatique about 2 months ago

Automatic expiration

Also available in: Atom PDF