betagouv / analyse-flux-insertion

Outil d'analyse des flux et échanges de données dans le domaine de l'insertion
2 stars 1 forks source link

Feat/Ardennes => Eviter d'enregistrer des NIR dans RDV-Solidarités #83

Closed qblanc closed 3 years ago

qblanc commented 3 years ago

Cette PR introduit un check au moment de la création de l'utilisateur sur RDV-Solidarités pour éviter d'y stocker des NIR.

aminedhobb commented 3 years ago

Il nous a dit que les numéros CAF pouvaient avoir différentes longueurs parfois, mais les NIR, c'est toujours 13 (et 15 avec la clé), donc a priori on est bon

Mais du coup est-ce que le numéro CAF peut être de longueur 13 ou 15 ? Si c'est le cas on le filtrerait faussement avec cette méthode 😶.

qblanc commented 3 years ago

Il nous a dit que les numéros CAF pouvaient avoir différentes longueurs parfois, mais les NIR, c'est toujours 13 (et 15 avec la clé), donc a priori on est bon

Mais du coup est-ce que le numéro CAF peut être de longueur 13 ou 15 ? Si c'est le cas on le filtrerait faussement avec cette méthode 😶.

Non, il est beaucoup plus court normalement, je cible spécifiquement le nombre de caractères du NIR pour éviter de filtrer des références valides. Mais de toute façon, il n'y a pas 10 000 solutions. On pourrait éventuellement ajouter une vérif que le numéro commence pour un 1 ou un 2 mais cela ne se serait pas parfait non plus. En outre, ce numéro CAF ne nous sert pour l'instant pas à grand chose, donc si certains sont filtrés à tort, ce n'est pas bien grave (beaucoup moins grave que si le filtre rate des NIR et les inscrits en base).