Projet

Général

Profil

Bug #84666

authntel activée mais pas de champ téléphone, KeyError: 'phone' lors de la réinitialisation de mot de passe

Ajouté par Sentry Io il y a 5 mois. Mis à jour il y a 5 mois.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Catégorie:
-
Version cible:
-
Début:
13 décembre 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

https://sentry.entrouvert.org/entrouvert/publik/issues/117328/

Le code fait :

    def clean(self):
        if self.authenticator.is_phone_authn_active:
            if (
                not self.cleaned_data['email']
                and not self.cleaned_data.get('email_or_username')
                and not self.cleaned_data['phone']
            ):

a priori quand l'authentification via le numéro est activée il devrait y avoir un champ téléphone mais visiblement pas ici.

KeyError: 'phone'
(7 additional frame(s) were not displayed)
...
  File "django/forms/forms.py", line 175, in is_valid
    return self.is_bound and not self.errors
  File "django/forms/forms.py", line 170, in errors
    self.full_clean()
  File "django/forms/forms.py", line 373, in full_clean
    self._clean_form()
  File "django/forms/forms.py", line 400, in _clean_form
    cleaned_data = self.clean()
  File "authentic2/forms/passwords.py", line 110, in clean
    and not self.cleaned_data['phone']

Révisions associées

Révision 1aa7e161 (diff)
Ajouté par Paul Marillonnet il y a 5 mois

forms: do not assume identifier presence at password reset (#84666)

Révision 42e39203 (diff)
Ajouté par Paul Marillonnet il y a 5 mois

translation update (#84666)

Historique

#1

Mis à jour par Frédéric Péters il y a 5 mois

  • Projet changé de Suivi des traces à Authentic 2
#2

Mis à jour par Paul Marillonnet il y a 5 mois

  • Statut changé de Nouveau à En cours
  • Assigné à mis à Paul Marillonnet
#3

Mis à jour par Paul Marillonnet il y a 5 mois

Étrange, je conçois mal comment ça aurait pu survenir :/
Peut-être une authentification tél activée entre le moment de la présentation du formulaire de mot de passe et de sa validation (?)
Je peux patcher mais aucune idée de comment tester cela…

#4

Mis à jour par Robot Gitea il y a 5 mois

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

Paul Marillonnet (pmarillonnet) a ouvert une pull request sur Gitea concernant cette demande :

#6

Mis à jour par Paul Marillonnet il y a 5 mois

Paul Marillonnet a écrit :

Étrange, je conçois mal comment ça aurait pu survenir :/
Peut-être une authentification tél activée entre le moment de la présentation du formulaire de mot de passe et de sa validation (?)
Je peux patcher mais aucune idée de comment tester cela…

Ok, pigé grâce au descriptif de #84839 plus détaillé que ce qu’est en mesure de fournir Sentry : en tapant un numéro de tél bidon, dans un mauvais format, le champ de téléphone du formulaire ne l’accepte pas et la valeur n’est pas présente dans le Form.cleaned_data.

#7

Mis à jour par Robot Gitea il y a 5 mois

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

Paul Marillonnet (pmarillonnet) a commencé à travailler sur une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Robot Gitea il y a 5 mois

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

Mis à jour par Robot Gitea il y a 5 mois

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

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#10

Mis à jour par Robot Gitea il y a 5 mois

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

Paul Marillonnet (pmarillonnet) a mergé une pull request sur Gitea concernant cette demande :

#11

Mis à jour par Transition automatique il y a 5 mois

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

Mis à jour par Transition automatique il y a 3 mois

Automatic expiration

Formats disponibles : Atom PDF