Bug #17920
gérer un timeout dans jsoncell
0%
Description
Quand on est sur une cellule jsoncell qui ne répond pas, alors ça se termine toujours en "barre rouge" car la requête ajax est interrompue par le timeout du service combo (gunicorn)
Ca serait mieux de pouvoir donner un timeout inférieur à 30 secondes, et retourner un résultat "erreur" au contexte de la cellule qui décidera ce qu'elle veut afficher
Fichiers
Révisions associées
json cell: add timeout migration (#17920)
Historique
Mis à jour par Thomas Noël il y a plus de 6 ans
- Fichier 0001-json-cell-add-timeout-parameter-17920.patch 0001-json-cell-add-timeout-parameter-17920.patch ajouté
- Patch proposed changé de Non à Oui
Ça ajoute par ricochet une gestion des erreurs des requêtes.
à discuter :- un timeout à 28s par défaut
- j'ai autorisé un timeout sur le mode "prototype" pour le passer à 2 secondes
- j'invente le status http "-1" en cas d'erreur de requête
Mis à jour par Thomas Noël il y a plus de 6 ans
- Statut changé de Nouveau à En cours
A noter que le timeout pourrait venir d'un settings, qui lui même pourrait venir d'une variable d'environnement (COMBO_WSGI_TIMEOUT, sur le modèle de ce qui a été fait dans Passerelle)
Mis à jour par Frédéric Péters il y a plus de 6 ans
- Statut changé de En cours à Résolu (à déployer)
Pas tout à fait enthousiaste (parce que les exceptions, je sais qu'elles nous arrivent par email, alors que le logger.warning, je crains qu'il ne se perde).
Mais poussé en corrigeant
logger.warning(u'error on request %r: %', url, unicode(e))
et
logger.debug(u'error on request %r: %', url, unicode(e))
où il manquait une lettre derrière le second %.
commit 2d394d02217ef71e0e3d8a0f63f2fcf66acaeace Author: Thomas NOEL <tnoel@entrouvert.com> Date: Mon Aug 7 16:35:30 2017 +0200 json cell: add timeout parameter (#17920) ... and handle requests errors, consequently
Mis à jour par Frédéric Péters il y a plus de 5 ans
- Statut changé de Résolu (à déployer) à Solution déployée
json cell: add timeout parameter (#17920)
... and handle requests errors, consequently