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 la sanitation des noms de RIBs #365

Closed
JeSuisUnCaillou opened this issue Apr 30, 2024 · 2 comments
Closed

Améliorer la sanitation des noms de RIBs #365

JeSuisUnCaillou opened this issue Apr 30, 2024 · 2 comments

Comments

@JeSuisUnCaillou
Copy link

JeSuisUnCaillou commented Apr 30, 2024

Lorsque les utilisateurs saisissent un nom de RIB, ils copient collent parfois des caractères étranges.
On les retire au moment de créer le fichier XML qui sera envoyé à l'ASP, car ils ont une liste assez stricte de ce qui peut être envoyé.

Tout se passe ici : https://github.com/betagouv/aplypro/blob/main/app/services/asp/mappers/coord_paie_mapper.rb

Lorsqu'un crée un fichier XML pour l'ASP avec un mauvais caractère dans le RIB, on obtient l'erreur de cette ligne : https://github.com/betagouv/aplypro/blob/main/app/services/asp/mappers/coord_paie_mapper.rb/#L49
Et ça empêche la création complète du fichier XML (bloquant donc 7000 payments)

@JeSuisUnCaillou
Copy link
Author

JeSuisUnCaillou commented May 7, 2024

Au lieu de lever une erreur, il faudrait que la sanitation s'assure vraiment qu'on bien ait que les caractères autorisés avec une whitelist.

@freesteph
Copy link
Contributor

La seule subtilité qui empêche de faire un simple bitmask avec une liste de caractères autorisés c'est que certains doivent être remplacés par un espace et d'autre sans espaces :

https://github.com/betagouv/aplypro/blob/93ab402950ad97b5412f07688f8164870dd79bec/app/services/asp/mappers/coord_paie_mapper.rb#L14-L15

@pskl pskl closed this as completed May 17, 2024
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

No branches or pull requests

3 participants