Projet

Général

Profil

Development #74397

base-adresse : retenter une recherche en cas de 50x

Ajouté par Thomas Noël il y a environ un an. Mis à jour il y a environ un an.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
10 février 2023
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Non
Planning:
Non

Description

On a pas mal de retour 500, 502, 504... lors des appels à https://api-adresse.data.gouv.fr/search

En testant à la main, on arrive à en avoir, et ce n'est qu'anecdotique, en général la requête suivante passe. Sans doute un pépin d'infra duale chez eux, ou que sais-je.

Je propose qu'on re-tente 2 fois lorsqu'on reçoit une 50x (si la réponse revient en moins d'une seconde, pour éviter du lag sur l'autocomplétion).

(note : on a déjà un max_retries, mais ça ne concerne que les pépins de connexion, ici la connexion se passe bien, c'est juste que la BAN nous répond une 50x)

(note 2 : ça a un peu de lien avec #72914 sauf que je préfère retenter au lieu de crasher aussitôt)

Révisions associées

Révision b5995828 (diff)
Ajouté par Benjamin Dauvergne il y a environ un an

base_adresse: retry on 504 and reduce to 3 short retries (#74397)

Historique

#1

Mis à jour par Benjamin Dauvergne il y a environ un an

  • Assigné à mis à Benjamin Dauvergne
#2

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Nouveau à Solution proposée

Benjamin Dauvergne (bdauvergne) a ouvert une pull request sur Gitea concernant cette demande :

#3

Mis à jour par Benjamin Dauvergne il y a environ un an

En fait c'est déjà géré il suffit de le configurer.

#4

Mis à jour par Benjamin Dauvergne il y a environ un an

Yapuka faire des tests.

#5

Mis à jour par Benjamin Dauvergne il y a environ un an

On ne peut pas tester le comportement pour les ConnectionTimeout/ReadError ça n'est pas supporté par responses.

#6

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution proposée à Solution validée

Serghei Mihai (smihai) a approuvé une pull request sur Gitea concernant cette demande :

#7

Mis à jour par Robot Gitea il y a environ un an

  • Statut changé de Solution validée à Résolu (à déployer)

Benjamin Dauvergne (bdauvergne) a mergé une pull request sur Gitea concernant cette demande :

#8

Mis à jour par Transition automatique il y a environ un an

  • Statut changé de Résolu (à déployer) à Solution déployée
#9

Mis à jour par Transition automatique il y a 11 mois

Automatic expiration

Formats disponibles : Atom PDF