Bug #84666
authntel activée mais pas de champ téléphone, KeyError: 'phone' lors de la réinitialisation de mot de passe
0%
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']
Historique
Mis à jour par Paul Marillonnet il y a 5 mois
- Statut changé de Nouveau à En cours
- Assigné à mis à Paul Marillonnet
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…
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/216
- Titre : forms: do not assume identifier presence at password reset (#84666)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/216/files
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
.
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/216
- Titre : WIP: forms: do not assume identifier presence at password reset (#84666)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/216/files
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 :
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 :
- URL : https://git.entrouvert.org/entrouvert/authentic/pulls/216
- Titre : forms: do not assume identifier presence at password reset (#84666)
- Modifications : https://git.entrouvert.org/entrouvert/authentic/pulls/216/files
Mis à jour par Transition automatique il y a 5 mois
- Statut changé de Résolu (à déployer) à Solution déployée
forms: do not assume identifier presence at password reset (#84666)