Projet

Général

Profil

Bug #25327

django.core.exceptions.ImproperlyConfigured: LDAP_AUTH_SETTINGS: attribute 'groupsu' must be a list or a tuple

Ajouté par Frédéric Péters il y a presque 6 ans. Mis à jour il y a plus de 5 ans.

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

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

Comme #25303 mais pour un autre attribut.

Et vraisemblablement derrière la même chose aussi sur groupstaff. Et groupactive.

    _TO_ITERABLE = ('url', 'groupsu', 'groupstaff', 'groupactive')
...
        for i in cls._TO_ITERABLE:
            if isinstance(block[i], six.string_types):
                block[i] = (block[i],)

Fichiers

Révisions associées

Révision d11d86bb (diff)
Ajouté par Frédéric Péters il y a presque 6 ans

backends/ldap: don't set a default value for group{su,staff,active} (#25327)

Révision 6235bc17 (diff)
Ajouté par Benjamin Dauvergne il y a presque 6 ans

backends/ldap: convert to iterable before type checking (#25327)

Revert d11d86bb50f7.

Historique

#1

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

Le même patch que dans #23698 (absolument pas testé) mais peut-être qu'on voudrait plutôt modifier ce bout pour en exclure _TO_ITERABLE : ?

                if (isinstance(cls._DEFAULTS[d], (list, tuple)) and.
                        not isinstance(block[d], (list, tuple))):
                    raise ImproperlyConfigured(
                        'LDAP_AUTH_SETTINGS: attribute %r must be a list or a tuple' % d)
#2

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

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

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

  • Statut changé de Solution validée à Résolu (à déployer)
commit d11d86bb50f7c7987a81d7cf969cb8251760dc51
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Tue Jul 17 21:43:02 2018 +0200

    backends/ldap: don't set a default value for group{su,staff,active} (#25327)
#4

Mis à jour par Benjamin Dauvergne il y a presque 6 ans

  • Statut changé de Résolu (à déployer) à En cours
  • Assigné à mis à Benjamin Dauvergne

Bon le patch ne marche pas.

#5

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

  • Statut changé de En cours à Résolu (à déployer)
commit 6235bc17829f3ff185948dd340178d7fef4df192
Author: Benjamin Dauvergne <bdauvergne@entrouvert.com>
Date:   Wed Jul 18 11:22:51 2018 +0200

    backends/ldap: convert to iterable before type checking (#25327)

    Revert d11d86bb50f7.
#6

Mis à jour par Benjamin Dauvergne il y a plus de 5 ans

  • Statut changé de Résolu (à déployer) à Fermé

Formats disponibles : Atom PDF