Project

General

Profile

Development #21978

endpoint d'info sur la disponibilité

Added by Frédéric Péters over 1 year ago. Updated about 1 month ago.

Status:
Solution déployée
Priority:
Normal
Target version:
-
Start date:
19 Feb 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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.

0001-misc-avoid-circular-imports-21978.patch View (2.03 KB) Lauréline Guerin, 11 Oct 2019 02:37 PM

0002-base-resource-add-automatic-up-endpoint-21978.patch View (5.47 KB) Lauréline Guerin, 11 Oct 2019 02:37 PM

0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch View (1.34 KB) Lauréline Guerin, 11 Oct 2019 02:37 PM

0002-base-resource-add-automatic-up-endpoint-21978.patch View (6.2 KB) Lauréline Guerin, 11 Oct 2019 03:08 PM

0003-solis-sp_fr-add-fixme-to-deprecate-ping-endpoints-21.patch View (1.34 KB) Lauréline Guerin, 11 Oct 2019 03:08 PM

0001-misc-avoid-circular-imports-21978.patch View (2.03 KB) Lauréline Guerin, 11 Oct 2019 03:08 PM


Related issues

Related to Passerelle - Support #36838: astregs : ajouter un appel ws isalive. Rejeté 10 Oct 2019

Associated revisions

Revision f3c7d04f (diff)
Added by Lauréline Guérin about 1 month ago

misc: avoid circular imports (#21978)

Revision c2eee211 (diff)
Added by Lauréline Guérin about 1 month ago

base resource: add automatic 'up' endpoint (#21978)

if method 'check_status' is defined, provide a 'up' endpoint

Revision 837facd4 (diff)
Added by Lauréline Guérin about 1 month ago

solis, sp_fr: add fixme to deprecate ping endpoints (#21978)

History

#1 Updated by Lauréline Guerin about 1 month ago

  • Assignee set to Lauréline Guerin

#2 Updated by Lauréline Guerin about 1 month ago

#3 Updated by Frédéric Péters about 1 month ago

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.).

#5 Updated by Benjamin Dauvergne about 1 month ago

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à.

#6 Updated by Frédéric Péters about 1 month ago

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.

#7 Updated by Mikaël Ates about 1 month ago

  • Related to Support #36838: astregs : ajouter un appel ws isalive. added

#9 Updated by Benjamin Dauvergne about 1 month ago

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.

#10 Updated by Frédéric Péters about 1 month ago

  • Status changed from Solution proposée to Solution validée

Allons-y donc ainsi.

#11 Updated by Lauréline Guerin about 1 month ago

  • Status changed from Solution validée to 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)

#12 Updated by Frédéric Péters about 1 month ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF