Development #21978
endpoint d'info sur la disponibilité
0%
Description
Les connecteurs qui implémentent un suivi de la disponibilité pourraient proposer automatiquement un endpoint, ça permettrait par exemple dans un formulaire de faire une première page avec une condition d'entrée (not webservice.clicrdv.up) pour afficher un message comme quoi c'est pas le moment d'essayer de prendre un rendez-vous.
Fichiers
Demandes liées
Révisions associées
base resource: add automatic 'up' endpoint (#21978)
if method 'check_status' is defined, provide a 'up' endpoint
solis, sp_fr: add fixme to deprecate ping endpoints (#21978)
Historique
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Fichier 0001-misc-avoid-circular-imports-21978.patch 0001-misc-avoid-circular-imports-21978.patch ajouté
- Fichier 0002-base-resource-add-automatic-up-endpoint-21978.patch 0002-base-resource-add-automatic-up-endpoint-21978.patch ajouté
- Fichier 0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch 0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Frédéric Péters il y a plus de 4 ans
Je baserais plutôt ça sur le retour de get_availability_status, sur l'idée d'obtenir ainsi une réponse immédiate (alors que check_status va généralement se trouver faire une requête http, qui peut terminer en timeout, etc.).
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Fichier 0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch 0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch ajouté
- Fichier 0002-base-resource-add-automatic-up-endpoint-21978.patch 0002-base-resource-add-automatic-up-endpoint-21978.patch ajouté
- Fichier 0001-misc-avoid-circular-imports-21978.patch 0001-misc-avoid-circular-imports-21978.patch ajouté
en utilisant la méthode 'down' cette fois
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
De mon coté je ne vois pas l'intérêt d'un endpoint up qui ne test pas si c'est up, on a vraiment d'une information temps réel (enfin sur les cas que je vois au CD06); donc ok pour dire down si c'est déjà down (dans l'idée qu'il ne sert à rien de mitrailler un service qui a déjà du mal) sans vérifier mais si ça dit up j'aimerai que ce soir sûr à ce moment là.
Mis à jour par Frédéric Péters il y a plus de 4 ans
Moi j'aime bien ainsi, parce que je dirais qu'on a plutôt besoin d'une information retournée immédiatement, parce que si utiliser cet endpoint ajoute ne fut-ce qu'une demi seconde au rendu d'une démarche, on n'utilisera pas.
Mis à jour par Mikaël Ates il y a plus de 4 ans
- Lié à Support #36838: astregs : ajouter un appel ws isalive. ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 4 ans
Frédéric Péters a écrit :
Moi j'aime bien ainsi, parce que je dirais qu'on a plutôt besoin d'une information retournée immédiatement, parce que si utiliser cet endpoint ajoute ne fut-ce qu'une demi seconde au rendu d'une démarche, on n'utilisera pas.
Ok. De toute façon je pense que dans un cas super instable comme AstreGS il faut appeler un endpoint particulier (le truc est up mais certains endpoint ne marchent pas des fois, sans raison), ce up serait de toute façon trop imprécis.
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Solution proposée à Solution validée
Allons-y donc ainsi.
Mis à jour par Lauréline Guérin il y a plus de 4 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 837facd4b2def5002ab342c357d70addb1588a9b Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Oct 11 14:36:33 2019 +0200 solis, sp_fr: add fixme to deprecate ping endpoints (#21978) commit c2eee211eb28232e7ac4c63b2a29994de3285e7e Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Oct 11 14:24:00 2019 +0200 base resource: add automatic 'up' endpoint (#21978) if method 'check_status' is defined, provide a 'up' endpoint commit f3c7d04f5d66c4887baea3acd60d2d2ed1a517c4 Author: Lauréline Guérin <zebuline@entrouvert.com> Date: Fri Oct 11 14:23:27 2019 +0200 misc: avoid circular imports (#21978)
Mis à jour par Frédéric Péters il y a plus de 4 ans
- Statut changé de Résolu (à déployer) à Solution déployée
misc: avoid circular imports (#21978)