Projet

Général

Profil

Bug #23514

correction utilisation app_settings.A2_ACCEPT_EMAIL_AUTHENTICATION

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

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

100%

Temps estimé:
Patch proposed:
Oui
Planning:

Description

/tmp/tox-jenkins/authentic/coverage-dj18-authentic-pg/local/lib/python2.7/site-packages/django/contrib/auth/__init__.py:74: in authenticate
    user = backend.authenticate(**credentials)
src/authentic2/backends/ldap_backend.py:339: in authenticate
    config = self.get_config()
src/authentic2/backends/ldap_backend.py:331: in get_config
    cls.update_default(block)
src/authentic2/backends/ldap_backend.py:1096: in update_default
    if d == 'user_filter' and app_settings.A2_ACCEPT_EMAIL_AUTHENTICATION:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 

self = <authentic2.app_settings.AppSettings object at 0x7f99ba3e2790>
key = 'A2_ACCEPT_EMAIL_AUTHENTICATION'

    def __getattr__(self, key):
        if key not in self.defaults:
>           raise AttributeError('unknown key %s' % key)
E           AttributeError: unknown key A2_ACCEPT_EMAIL_AUTHENTICATION

src/authentic2/app_settings.py:59: AttributeError

Depuis #22349, https://jenkins.entrouvert.org/job/authentic2/2614/console

Fichiers

0001-misc-correctly-reference-ACCEPT_EMAIL_AUTHENTICATION.patch (1,06 ko) 0001-misc-correctly-reference-ACCEPT_EMAIL_AUTHENTICATION.patch Frédéric Péters, 30 avril 2018 22:34
0001-app_settings-rename-ACCEPT_EMAIL_AUTHENTICATION-to-A.patch (2,86 ko) 0001-app_settings-rename-ACCEPT_EMAIL_AUTHENTICATION-to-A.patch Benjamin Dauvergne, 01 mai 2018 00:42

Demandes liées

Lié à Authentic 2 - Development #22349: authent ldap via adresse emailFermé07 mars 2018

Actions

Révisions associées

Révision b140cb26 (diff)
Ajouté par Benjamin Dauvergne il y a environ 6 ans

app_settings: rename ACCEPT_EMAIL_AUTHENTICATION to A2_ACCEPT_EMAIL_AUTHENTICATION (fixes #23514)

Historique

#1

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

#2

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

#3

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

(de manière générale la mécanique app_settings me semble trop lourde/exigeante/particulière, mais ce serait toute une affaire de la retirer, je ne crée pas de ticket)

#4

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

Je préfère qu'on renomme les anciens usages pour aller vers le nommage commun.

#5

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

Si personne ne l'utilisait, ok. (j'ai juste vérifié notre SaaS)

#6

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

Le setting s'appelait quand même A2_ACCEPT_EMAIL_AUTHENTICATION, mais si tu parles du app_settings lui même, je ne pense pas qu'il y avait des usages hors du code d'authentic.

#7

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

  • Statut changé de En cours à Résolu (à déployer)
  • % réalisé changé de 0 à 100
#8

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

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

Formats disponibles : Atom PDF