Development #28643
Pas de persistance des cookies lors de la vérification de la disponibilité ou dans un shell
0%
Description
Que j'aurais pu aussi intituler "planitech : check_status innopérant".
On a des cookies depuis #27654, mais vu l'implémentation, pas de persistance cookie lors des appels à availability depuis un cron ou un shell, le check_status de planitech essaie de s'authentifier (et il a besoin de cookie), boum.
J'hésite entre assurer la persistance des cookies dans cette situation ou revoir le check_status de planitech (qui pourrait juste commencer la danse d'authentification sans la finir, et considérer que ça suffit comme ça pour dire que le service est 'up').
Fichiers
Demandes liées
Révisions associées
Historique
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Je dirai d'assurer la persistance des cookies dans availability() si cookiejar n'est pas là, crée le.
Mis à jour par Thomas Noël il y a plus de 5 ans
J'ai en parallèle dit oralement à Emmanuel que le check_status pouvait aussi se limiter à un check minimal (planitech "à l'air d'être là"). Parce qu'à l'usage les cas de panne, à 99%, c'est le réseau HS, la machine éteinte, du 404/500...
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
On fait bien ce qu'on veut pour planitech, mais dans l'absolu quand on écriera d'autre check_status() c'est bien de se dire qu'on reste dans les mêmes conditions que celles d'un endpoints, sinon ça devient compliqué pour rien.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Tracker changé de Bug à Development
- Sujet changé de Pas de persitance des cookies lors de la vérifiaction de la disponiblité à Pas de persitance des cookies lors de la vérifiaction de la disponiblité ou dans un shell
Dans un shell pour faire mumuse avec son connecteur, c'est la même histoire.
Et je ressort https://dev.entrouvert.org/issues/27654#note-10 de mon chapeau, qui nous aurait évité ce ticket.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Lié à Bug #28651: planitech: revoir le echeck_status ajouté
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Emmanuel Cazenave a écrit :
Dans un shell pour faire mumuse avec son connecteur, c'est la même histoire.
Et je ressort https://dev.entrouvert.org/issues/27654#note-10 de mon chapeau, qui nous aurait évité ce ticket.
Qu'est qui empêche de mettre l'implémentation de def session() à la place de def requests() ?
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Benjamin Dauvergne a écrit :
Emmanuel Cazenave a écrit :
Dans un shell pour faire mumuse avec son connecteur, c'est la même histoire.
Et je ressort https://dev.entrouvert.org/issues/27654#note-10 de mon chapeau, qui nous aurait évité ce ticket.
Qu'est qui empêche de mettre l'implémentation de
def session()à la place dedef requests()?
Ça viendrait contrarier #24619, mais son utilité semble discutable.
Mis à jour par Frédéric Péters il y a plus de 5 ans
Ça viendrait contrarier #24619, mais son utilité semble discutable.
Oui, on a depuis vu que la situation n'était pas identique à Combo (où la session se trouvait partagée entre requêtes).
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Fichier 0001-persist-cookies-on-a-connector-instance-28643.patch 0001-persist-cookies-on-a-connector-instance-28643.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Sujet changé de Pas de persitance des cookies lors de la vérifiaction de la disponiblité ou dans un shell à Pas de persistance des cookies lors de la vérification de la disponibilité ou dans un shell
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
Benjamin Dauvergne a écrit :
Pourquoi tu vires le test ?
Ou alors pourquoi t'en écris pas un autre...
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
Mis à jour par Benjamin Dauvergne il y a plus de 5 ans
- Statut changé de Solution proposée à Solution validée
- Assigné à mis à Emmanuel Cazenave
Ok.
Mis à jour par Emmanuel Cazenave il y a plus de 5 ans
- Statut changé de Solution validée à Résolu (à déployer)
commit 0b4465a3004d46f91f242d03c2aee00f19ab43b4
Author: Emmanuel Cazenave <ecazenave@entrouvert.com>
Date: Thu Dec 13 18:26:58 2018 +0100
persist cookies on a connector instance (#28643)
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
Mis à jour par Benjamin Dauvergne il y a environ 5 ans
- Statut changé de Solution déployée à Fermé
persist cookies on a connector instance (#28643)