Project

General

Profile

Development #28643

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

Added by Emmanuel Cazenave 6 months ago. Updated about 1 month ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
05 Dec 2018
Due date:
% Done:

0%

Patch proposed:
Yes
Planning:
No

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 View (3.85 KB) Emmanuel Cazenave, 13 Dec 2018 06:27 PM


Related issues

Related to Passerelle - Bug #28651: planitech: revoir le echeck_status Fermé 05 Dec 2018

Associated revisions

Revision 0b4465a3 (diff)
Added by Emmanuel Cazenave 5 months ago

persist cookies on a connector instance (#28643)

History

#1 Updated by Benjamin Dauvergne 6 months ago

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

#2 Updated by Thomas Noël 6 months ago

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 Updated by Benjamin Dauvergne 6 months ago

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 Updated by Emmanuel Cazenave 6 months ago

  • Tracker changed from Bug to Development
  • Subject changed from Pas de persitance des cookies lors de la vérifiaction de la disponiblité to 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 Updated by Emmanuel Cazenave 6 months ago

  • Related to Bug #28651: planitech: revoir le echeck_status added

#6 Updated by Benjamin Dauvergne 6 months ago

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 Updated by Emmanuel Cazenave 6 months ago

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 Updated by Frédéric Péters 6 months ago

Ç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 Updated by Emmanuel Cazenave 5 months ago

#10 Updated by Benjamin Dauvergne 5 months ago

  • Subject changed from Pas de persitance des cookies lors de la vérifiaction de la disponiblité ou dans un shell to Pas de persistance des cookies lors de la vérification de la disponibilité ou dans un shell

#11 Updated by Benjamin Dauvergne 5 months ago

Pourquoi tu vires le test ?

#12 Updated by Benjamin Dauvergne 5 months ago

Benjamin Dauvergne a écrit :

Pourquoi tu vires le test ?

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

#13 Updated by Emmanuel Cazenave 5 months ago

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 Updated by Benjamin Dauvergne 5 months ago

  • Status changed from Solution proposée to Solution validée
  • Assignee set to Emmanuel Cazenave

Ok.

#15 Updated by Emmanuel Cazenave 5 months ago

  • Status changed from Solution validée to 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 Updated by Frédéric Péters 5 months ago

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

#17 Updated by Benjamin Dauvergne about 1 month ago

  • Status changed from Solution déployée to Fermé

Also available in: Atom PDF