vaccin.click icon indicating copy to clipboard operation
vaccin.click copied to clipboard

Problèmes de mise à jour qui s'arrête

Open julienw opened this issue 3 years ago • 23 comments

Salut, j'ai l'impression que parfois les mises à jour s'arrêtent: Capture d’écran de 2021-05-25 18-44-38

Possible que ce soit parce que j'ai "pausé" puis redémarré? Mais les horaires sont assez différents tout de même. Une piste possible ?

julienw avatar May 25 '21 16:05 julienw

ah oui bizarre effectivement. Ton ordi ne s'est pas mis en veille par hasard ?

dunglas avatar May 25 '21 17:05 dunglas

Un ami me dit que quand on met l'ordi en veille, l'extension ne recommence pas à vérifier automatiquement. cc @pirquessa Peut-être que l'on devrait repasser à l'API fournie par browser.alarm pour être sûr que la tâche de fond se relance ?

dunglas avatar May 25 '21 17:05 dunglas

Ce qui est étrange c'est que tu en ais autant de jaune. C'est que les checks ne sont pas finis... Les jobs s'exécutent comme ils sont rentrés dans la queue, pas forcément dans l'ordre de la UI:

  • tu ajoutes A, B, C
  • le traitement commence avec A, fini puis B => QUEUE = C, A
  • tu ajoutes D, E => QUEUE = C, A, D, E
  • B fini => QUEUE = C, A, D, E, B
  • ...

pirquessa avatar May 25 '21 17:05 pirquessa

browser.alarm, je connais pas, je vais regarder ca.

pirquessa avatar May 25 '21 17:05 pirquessa

ah oui bizarre effectivement. Ton ordi ne s'est pas mis en veille par hasard ?

Non, pas aujourd'hui ! En revanche écran locké (je me suis toujours demandé si l'extension s'arrêtait dans ce cas... je n'ai pas vérifié dans le code j'avoue :-) )

Je ne connais pas browser.alarm... pas trop le temps de regarder de plus près pour l'instant :/

Ce qui est étrange c'est que tu en ais autant de jaune.

C'est aussi ce que je me dis :-) À un moment on devrait timeout et recharger la page... normalement chaque waitForSelector doit timeouter donc c'est vraiment curieux.

julienw avatar May 25 '21 17:05 julienw

Un ami me dit que quand on met l'ordi en veille, l'extension ne recommence pas à vérifier automatiquement. cc @pirquessa Peut-être que l'on devrait repasser à l'API fournie par browser.alarm pour être sûr que la tâche de fond se relance ?

La doc est vite vue, je ne vois pas ou tu veux en venir.

https://developer.mozilla.org/fr/docs/Mozilla/Add-ons/WebExtensions/API/idle peut être pour détecter un lock ?

pirquessa avatar May 25 '21 17:05 pirquessa

Ah oui ça devrait le faire aussi.

dunglas avatar May 25 '21 17:05 dunglas

Il faudrait d'abord valider que c'est bien le souci. J'ai redémarré Firefox et je vais voir si ça recommence et essayer de relier à des trucs que je fais.

julienw avatar May 25 '21 17:05 julienw

Salut, J'ai un problème similaire. Je n'ai qu'un seul centre qui se met à jour, dès le démarrage de Firefox (Developer 89.0b15 64bits)

L'extension affiche "<Centre de vaccincation> - Début de la vérification" une seule fois mais ne va pas plus loin et ne recommence pas de vérification.

Et dans les logs de l'extension, j'ai un message (exemple avec le centre de Pithiviers - 45) "Un en-tête X-Frame-Options non valide a été trouvé lors du chargement de « https://www.doctolib.fr/vaccination-covid-19/pithiviers/centre-de-vaccination-covid-de-pithiviers?highlight%5Bspeciality_ids%5D%5B%5D=5494 » : « ALLOWALL » n’est pas une directive valide." Je me dis que c'est peut être en lien avec le problème

J'ai aussi vu dans l'inspecteur réseau qu'il téléchargeais bien le json du centre mais pas le fichier availabilities.json de ce même centre

Valdaria avatar May 25 '21 19:05 Valdaria

"Un en-tête X-Frame-Options non valide a été trouvé lors du chargement de « https://www.doctolib.fr/vaccination-covid-19/pithiviers/centre-de-vaccination-covid-de-pithiviers?highlight%5Bspeciality_ids%5D%5B%5D=5494 » : « ALLOWALL » n’est pas une directive valide."

Ce message est "normal" (c'est dû à Doctolib) et n'empêche pas le fonctionnement de l'extension. Il n'y en a pas d'autre ?

dunglas avatar May 25 '21 19:05 dunglas

J'ai essayé de mon côté avec le centre de Pithiviers et ça semble fonctionner correctement.

julienw avatar May 25 '21 19:05 julienw

C'est pas une solution, mais est-ce que désactiver l'extension et la réactiver relance les checks ?

image

pirquessa avatar May 25 '21 19:05 pirquessa

Ce message est "normal" (c'est dû à Doctolib) et n'empêche pas le fonctionnement de l'extension. Il n'y en a pas d'autre ?

J'ai des warnings de cookies SameSite mais pas d'erreurs différentes

Désactiver et réactiver l'extension relance les checks mais bloque toujours au même point

Valdaria avatar May 25 '21 19:05 Valdaria

Tu peux nous donner l'url des centres que tu surveilles qu'on essaie de reproduire ca ?

pirquessa avatar May 25 '21 19:05 pirquessa

Je surveille ceux là Il n'y a que le troisième qui fonctionne, celui de Montargis

  • https://www.doctolib.fr/vaccination-covid-19/pithiviers/centre-de-vaccination-covid-de-pithiviers?highlight%5Bspeciality_ids%5D%5B%5D=5494
  • https://www.doctolib.fr/vaccination-covid-19/montargis/centre-de-vaccination-prefecture-du-loiret?highlight%5Bspeciality_ids%5D%5B%5D=5494
  • https://www.doctolib.fr/hopital-public/montargis/centre-de-vaccination-covid-ch-de-montargis?highlight%5Bspeciality_ids%5D%5B%5D=5494
  • https://www.doctolib.fr/vaccination-covid-19/chateauneuf-sur-loire/centre-de-vaccination-chateauneuf-sur-loire?highlight%5Bspeciality_ids%5D%5B%5D=5494

Valdaria avatar May 25 '21 19:05 Valdaria

Je viens de tester sur un firefox fraichement installé et tous les centres fonctionnent. J'ai donc supprimé et remis les mêmes centres sur mon firefox developer et ça refonctionne.

Je pense donc que ça viens du fait que je les ait ajouté avant le passage au package distribué sur addons.firefox.com et que ça ait conservé les cookies, car je n'ai pas eu à les remettre quand j'ai installé l'addon via firefox

Valdaria avatar May 25 '21 20:05 Valdaria

Hum, très étrange :/ Si tu ne reproduis plus c'est peut etre un soucis de format de données... (entre celles que tu avais avant ta dernière version & la dernière version. Genre il récup les données et plante parce qu'un attribut n'existe pas...

pirquessa avatar May 25 '21 20:05 pirquessa

pour info, j'ai locké mon PC et aussi suspend, mais je n'ai pas eu de souci, c'est reparti comme en 40 ce matin.

julienw avatar May 26 '21 08:05 julienw

https://github.com/dunglas/vaccin.click/pull/48 met un coup d'ordre sur le clear total des données, il y avait de petits soucis je pense. Faudrait voir si ca marche mieux avec :)

pirquessa avatar May 26 '21 08:05 pirquessa

Idée: peut-être que ça arrive quand on ouvre l'onglet "manuellement" et qu'on le referme à un moment clé ?

En tout cas j'ai eu le cas à nouveau ce matin, et en ouvrant les onglets "bloqués" ça les a débloqués en terminant le process.

julienw avatar May 26 '21 11:05 julienw

Ha, je vais regarder ca, c'est vrai que quand tu ouvres à la main on envoie quand même les messages de processing, ca fou peut être le bordel quand la queue 👍

pirquessa avatar May 26 '21 12:05 pirquessa

Une autre idée est peut-être lorsque je garde un des centres ouverts en tab en fond...

Mais je n'ai pas encore réussi à reproduire le souci en essayant.

julienw avatar May 26 '21 16:05 julienw

Je viens de changer un peu l'algo dans https://github.com/dunglas/vaccin.click/pull/48: on remet un job en queue dès qu'on en lance un plutôt qu'attendre son retour. Comme ca, même si on a pas de retour, on aura bien un check suivant...

On ne dépend donc plus des onglets ouverts manuellement.

pirquessa avatar May 26 '21 20:05 pirquessa