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.

Sign up for GitHub

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

Jump to bottom

Ne sauvegarder le profil que lorsque l'utilisateur·rice l'a demandé #6

Open
DavidLibeau opened this issue Apr 6, 2020 · 2 comments · May be fixed by #10
Open

Ne sauvegarder le profil que lorsque l'utilisateur·rice l'a demandé #6

DavidLibeau opened this issue Apr 6, 2020 · 2 comments · May be fixed by #10

Comments

@DavidLibeau
Copy link

DavidLibeau commented Apr 6, 2020 •
edited

Le formulaire sauvegarde ses données dans le stockage local du navigateur pour que l'utilisateur·rice n'ait pas a renseigner à nouveau l'intégralité des informations lorsqu'iel souhaite générer un pdf. Cette fonctionnalité est très pratique. Cependant, je pense qu'il serait souhaitable de demander l'avis de l'utilisateur·rice avant d'enregistrer toute donnée.

Ainsi, il faudrait que la fonction saveProfile() ne soit exécutée que lorsque qu'une case à cocher soit validée.

Par exemple :

[ ] Enregistrer mes informations dans mon navigateur web pour ne pas à avoir les renseigner la prochaine fois
@DavidLibeau DavidLibeau linked a pull request Apr 6, 2020 that will close this issue
Open
@JohnXLivingston
Copy link

D'après ce que j'ai constaté, il n'est pas sauvegardé à cette fin. En réalité, ce qui est stocké n'est jamais lu après coup (en tout cas dans la version qui était en prod ce 6 avril vers 17h). La fonction getProfile n'est appelée que dans le submit du formulaire, juste après le save...
Je pense que c'est un reste de code, parce qu'ils avaient effectivement dû prévoir de restaurer les données, mais que finalement ils ne l'ont pas fait.

J'ai signalé le bug (sur twitter), et on m'a dit qu'il serait corrigé rapidement.

À noter qu'il pose - à mon sens - un grave problème de sécurité. Si vous générez le pdf sur un terminal qui n'est pas le votre, les données restent. Si vous le faite sur votre smartphone et qu'on vout le vole, on peut vous voler votre état civil. Etc.

@DavidLibeau
Copy link
Author

Effectivement, je n'ai pas eu le temps de faire une issue sur ce sujet mais il semblerait que les données soient stockées mais jamais utilisées.

@JohnXLivingston JohnXLivingston mentioned this issue Apr 7, 2020
Closed
@DavidLibeau DavidLibeau mentioned this issue Apr 9, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Demander avant de sauvegarder les données en local
2 participants
@DavidLibeau @JohnXLivingston