betagouv / mon-aide-cyber

Apache License 2.0
7 stars 2 forks source link

[CORRECTION] Corrige le problème de traitement asynchrone lors de la consommation d’événements par le bus d’événements #497

Closed bbougon closed 3 months ago

bbougon commented 3 months ago

Contexte

Import des Aidants

Reproduction

Résultat constaté

Le nombre d’événements AIDANT_CREE dans Metabase ne correspond pas au nombre de comptes Aidants réellement créés

Exécution de l’import d’Aidants

Capture d’écran 2024-06-07 à 10 15 23

Aidants créés dans MAC

Capture d’écran 2024-06-07 à 10 16 21

Nombre d’événements AIDANT_CREE dans Metabase

Capture d’écran 2024-06-07 à 10 16 03

Résultat attendu

Le nombre d’événements AIDANT_CREE dans Metabase doit correspondre aux nombre de comptes effectivement créés.

bbougon commented 3 months ago

🐛 Le problème

Lorsque l’on utilise nos scripts d’administration, on fait un npm run… qui a pour effet de lancer un processus node. Ce processus est arrêté dès lors que le script renvoie un signal de sortie. forEach ne gère pas l’asynchronisme il faut le remplacer par un for…of.

NB : dans l’exemple ci-dessus, nous importons 791 Aidants, nous avons déjà 1 Aidant précédemment créé

✅ La solution

Exécution de l’umport d’Aidants :

Capture d’écran 2024-06-07 à 10 15 23

Aidants créés dans MAC

Capture d’écran 2024-06-07 à 10 16 21

Nombre d’événements AIDANT_CREE dans Metabase avec le correctif

Capture d’écran 2024-06-07 à 10 20 01