Projet

Général

Profil

Development #63580

logguer une erreur / interdire une config ldap qui n'aurait pas external_id_tuples

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

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

Sans ça il va y avoir création sans fin des même comptes.


Fichiers


Demandes liées

Lié à Authentic 2 - Bug #63609: ldap, si un external_id ne peut être construit, ne pas créer d'utilisateurRejeté05 avril 2022

Actions
Bloque Authentic 2 - Development #63646: ldap, ajouter une recherche via les attributs UUID implicites (entryuuid, objectguid ou nsuniqueid)Fermé06 avril 2022

Actions

Révisions associées

Révision 78e98c7a (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 ans

ldap: fail cleanly when an external_id cannot be made (#63580)

Révision a171dd85 (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 ans

ldap: fix sync_ldap_users tests (#63580)

Révision b6ab4b32 (diff)
Ajouté par Benjamin Dauvergne il y a environ 2 ans

translation update (#63580)

Historique

#2

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

En fait si pas défini, ça prend la valeur par défaut : 'external_id_tuples': (('uid',),('dn:noquote',),). Mais si pas d'uid retourné par l'annuaire, plouf pas d'identifiant généré, doublons etc.

Et à vue de nez sur 90% des raccordements on utilise "external_id_tuples": [["samaccountname"],["dn:noquote"]] donc je serais pour juste changer la valeur par défaut.

#3

Mis à jour par Frédéric Péters il y a environ 2 ans

  • Lié à Bug #63609: ldap, si un external_id ne peut être construit, ne pas créer d'utilisateur ajouté
#4

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Assigné à mis à Benjamin Dauvergne
#5

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

#6

Mis à jour par Emmanuel Cazenave il y a environ 2 ans

Pourrait on aussi avoir la configuration la plus commune mise par défaut, monsieur ?

#7

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

Ça a corrigé un petit bug dans les logs de sync-ldap-users, création confondu avec une mise à jour, qu'il fallu répercuter dans les tests, en les simplifiant au passage.

#8

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de Solution proposée à En cours
#9

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

Emmanuel Cazenave a écrit :

Pourrait on aussi avoir la configuration la plus commune mise par défaut, monsieur ?

Non ça risque forcément de casser quelque chose quelque part de changer la valeur par défaut.

#10

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de En cours à Solution proposée
#12

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Bloque Development #63646: ldap, ajouter une recherche via les attributs UUID implicites (entryuuid, objectguid ou nsuniqueid) ajouté
#13

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Rebasé sur le ticket d'ajout de l'authentification par email (#62270)
  • Correction d'un typo dans le message destiné aux utilisateurs
  • Ajout d'un commit pour la traduction (il est entendu que c'est autorisé si le commit est séparé des autres)
#14

Mis à jour par Serghei Mihai il y a environ 2 ans

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

Go.

#15

Mis à jour par Benjamin Dauvergne il y a environ 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit b6ab4b32aad8f6f30722f4083def6dd2b6582cd5
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Thu Apr 7 08:45:37 2022 +0200

    translation update (#63580)

commit a171dd85e9b08c46b83c81477e411a31ee773bda
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Apr 6 18:33:06 2022 +0200

    ldap: fix sync_ldap_users tests (#63580)

commit 78e98c7a7ad5bee18d2aa36deaba578f09692877
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Apr 6 17:22:30 2022 +0200

    ldap: fail cleanly when an external_id cannot be made (#63580)
#16

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

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

Mis à jour par Transition automatique il y a presque 2 ans

Automatic expiration

Formats disponibles : Atom PDF