Projet

Général

Profil

Development #28643

Pas de persistance des cookies lors de la vérification de la disponibilité ou dans un shell

Ajouté par Emmanuel Cazenave il y a 3 mois. Mis à jour il y a 2 mois.

Statut:
Solution déployée
Priorité:
Normal
Assigné à:
Version cible:
-
Début:
05 déc. 2018
Echéance:
% réalisé:

0%

Patch proposed:
Oui

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

0001-persist-cookies-on-a-connector-instance-28643.patch Voir (3,85 ko) Emmanuel Cazenave, 13 déc. 2018 18:27


Demandes liées

Lié à Passerelle - Bug #28651: planitech: revoir le echeck_status Solution déployée 05 déc. 2018

Révisions associées

Révision 0b4465a3 (diff)
Ajouté par Emmanuel Cazenave il y a 2 mois

persist cookies on a connector instance (#28643)

Historique

#1 Mis à jour par Benjamin Dauvergne il y a 3 mois

Je dirai d'assurer la persistance des cookies dans availability() si cookiejar n'est pas là, crée le.

#2 Mis à jour par Thomas Noël il y a 3 mois

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

#3 Mis à jour par Benjamin Dauvergne il y a 3 mois

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.

#4 Mis à jour par Emmanuel Cazenave il y a 3 mois

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

#5 Mis à jour par Emmanuel Cazenave il y a 3 mois

  • Lié à Bug #28651: planitech: revoir le echeck_status ajouté

#6 Mis à jour par Benjamin Dauvergne il y a 3 mois

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() ?

#7 Mis à jour par Emmanuel Cazenave il y a 3 mois

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 de def requests() ?

Ça viendrait contrarier #24619, mais son utilité semble discutable.

#8 Mis à jour par Frédéric Péters il y a 3 mois

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

#9 Mis à jour par Emmanuel Cazenave il y a 2 mois

#10 Mis à jour par Benjamin Dauvergne il y a 2 mois

  • 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

#11 Mis à jour par Benjamin Dauvergne il y a 2 mois

Pourquoi tu vires le test ?

#12 Mis à jour par Benjamin Dauvergne il y a 2 mois

Benjamin Dauvergne a écrit :

Pourquoi tu vires le test ?

Ou alors pourquoi t'en écris pas un autre...

#13 Mis à jour par Emmanuel Cazenave il y a 2 mois

De #27654 il reste encore tests/test_generic_endpoint.py ::test_endpoint_cookies.

A mon sens celui que je vire n'était pertinent que rapport à l'implémentation choisie dans #27654 et encore.

#14 Mis à jour par Benjamin Dauvergne il y a 2 mois

  • Assigné à mis à Emmanuel Cazenave
  • Statut changé de Solution proposée à Solution validée

Ok.

#15 Mis à jour par Emmanuel Cazenave il y a 2 mois

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

#16 Mis à jour par Frédéric Péters il y a 2 mois

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

Formats disponibles : Atom PDF