Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Améliorer l'expérience utilisateur en cas d'erreur explicite FranceConnect #506

Merged
merged 1 commit into from
Jan 25, 2022

Conversation

tut-tuuut
Copy link
Member

@tut-tuuut tut-tuuut commented Jan 21, 2022

🌮 Objectif

Quand FranceConnect nous retourne une erreur explicite, l'erreur n'est pas traitée côté AC et on se contente d'afficher un écran blanc avec un code 403. Il faut aller décoder l'URL pour savoir ce qui s'est passé :

Capture d’écran 2022-01-21 à 10 41 24

Par exemple ici on demande le scope "gender" et apparemment on ne devrait plus...

🔍 Implémentation

Je propose de réutiliser la fonction fc_error quand la requête contient un paramètre "error". Dans ce cas on logue l'erreur et redirige l'utilisateur vers la page de création de mandat avec un message gentil. (À défaut d'être très utile, certes ! On fait c'qu'on peut hein.)

Images

Le message explicite remonte dans Sentry et c'est cool :

Capture d’écran 2022-01-21 à 11 12 59

@tut-tuuut tut-tuuut force-pushed the ahr/make-it-more-user-friendly-when-fc-error branch from 4ce5c28 to 0a4743b Compare January 21, 2022 09:53
@tut-tuuut tut-tuuut force-pushed the ahr/make-it-more-user-friendly-when-fc-error branch from 0a4743b to cef8cfa Compare January 25, 2022 07:49
@tut-tuuut tut-tuuut merged commit 5fa17ce into main Jan 25, 2022
@tut-tuuut tut-tuuut deleted the ahr/make-it-more-user-friendly-when-fc-error branch January 25, 2022 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants