Project

General

Profile

Bug #69466

PasswordResetConfirmView : gérer correctement les exceptions PasswordChangeError

Added by Benjamin Renard 5 months ago. Updated about 2 months ago.

Status:
Information nécessaire
Priority:
Normal
Assignee:
Benjamin Renard
Category:
-
Target version:
-
Start date:
22 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Dans la vue PasswordChangeError, lorsqu'une exception PasswordChangeError est lévée au moment de l'enregistrement, l'application affiche une erreur 500. Le patch ci-joint corrige cela en prenant en compte ce type d'exception comme cela est fait dans la vue PasswordChangeView et affiche l'erreur directement dans le formulaire.

Files

0003-PasswordResetConfirmView-fix-handling-PasswordChange.patch (1.14 KB) 0003-PasswordResetConfirmView-fix-handling-PasswordChange.patch Benjamin Renard, 22 September 2022 06:00 PM
0001-PasswordResetConfirmView-fix-handling-PasswordChange.patch (4.82 KB) 0001-PasswordResetConfirmView-fix-handling-PasswordChange.patch Benjamin Renard, 10 November 2022 03:12 PM

History

#1

Updated by Benjamin Dauvergne 4 months ago

  • Assignee set to Benjamin Dauvergne
#2

Updated by Benjamin Dauvergne 4 months ago

  • Assignee changed from Benjamin Dauvergne to Benjamin Renard

J'ai rebasé, il faudrait un/des tests.

#3

Updated by Benjamin Dauvergne 4 months ago

  • Status changed from Nouveau to Information nécessaire
#4

Updated by Benjamin Renard 3 months ago

Benjamin Dauvergne a écrit :

J'ai rebasé, il faudrait un/des tests.

Voilà un nouveau patch avec deux tests dont un simulant une erreur LDAP (passage de la DB en read-only dans le reset final du mdp).

#5

Updated by Benjamin Renard about 2 months ago

Je me permets une petite relance sur le sujet : as-tu put regarder mon dernier patch ? Quand pense-tu qu'il pourrait être intégré ? Notre client nous relance sur le sujet.

Also available in: Atom PDF