etalab / rncs_worker_api_entreprise

API pour récupérer et mettre à disposition les données du Répertoire National du Commerce et des Société
https://entreprise.data.gouv.fr
MIT License
23 stars 13 forks source link

Fix la duplication de mise à jour quotidienne #153

Closed brindu closed 4 years ago

brindu commented 4 years ago

Contenu de la PR

EDIT : j'embarque la MAJ de sécurité proposée par dependabot

Closes #149, description en commentaire du commit.

C'est un fix rustine, le problème est ailleurs dans la logique : l'algorithme sélectionne les mises à jour à "load" en base à partir de la dernière qui a été importée (utilisation du retour de DBCurrentDate). C'est une erreur et il faudrait plutôt ne "load" en base que les mises à jour plus récentes que la dernière qui a été "loadée" et qui, pour une raison où une autre, n'a pas encore été importée (mise à jour précédente en erreur par exemple).

Faire ce changement implique un assez gros refactor, prendrait du temps, et toucherait à l'algorithme d'orchestration de l'import qui aujourd'hui fonctionne bien. Ce sera donc pour plus tard, le patch d'aujourd'hui est loin d'être horrible.