Création Cantine : réparer bug redis, timeout variable rate limiting

[qloridant]

Contexte

La première étape de la création du SIRET demande le SIRET d'une cantine. Dès que le SIRET est renseigné nous requêtons la base SIRENE pour récupèrer des infos sur la cantine.

Problème

Temps de chargement de cette requête sur les plateformes :

• prod : plusieurs dizaine de seconde
• staging/local : quasi instantanée

Solution

[qloridant]

Sur le redis de prod, on a une valeur anormalement haute pour notre SIRET_API_CALL_PER_MINUTE On devrait avoir max 31, et on a actuellement 19299

notre_url.clever-cloud.com:notre_port> GET PROD_SIRET_API_CALLS_PER_MINUTE
"19299"

Notre code ne réinitialise jamais notre valeur SIRET_API_CALLS_PER_MINUTE si jamais deux workers incrémentent en parallèle la valeur.

En l'état, je ne comprend pas l'utilisation de redis pour ce qui s'apparente à une simple variable globale

[qloridant]

En faite on ne réinitialise pas la valeur sur notre clé redis :

• voir doc méthode redis.set()
• TTL PROD_SIRET_API_CALLS_PER_MINUTE renvoi -1 (Key exists, but no associated expiration)

[qloridant]

@alemangui @raphodn @hfroot Finalement, mes tests en local montrent que le système actuel devrait fonctionner. Je suis un peu bloqué la dessus...

Les faits :

• en prod, la variable dans Redis n'a pas de timeout
• en local, cette même variable est bien affectée par le timout

[hfroot]

@qloridant est-ce que t'as testé sur staging/demo ? Si ça ne marche pas non plus p-e contactes clever cloud, leur support est souvent reactif et aidant.