Skip to content
This repository has been archived by the owner on Dec 26, 2022. It is now read-only.

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

Open
julienw opened this issue May 25, 2021 · 23 comments
Open

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

julienw opened this issue May 25, 2021 · 23 comments

Comments

@julienw
Copy link
Collaborator

julienw commented May 25, 2021

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 ?

@dunglas
Copy link
Owner

dunglas commented May 25, 2021

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

@dunglas
Copy link
Owner

dunglas commented May 25, 2021

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 ?

@pirquessa
Copy link
Collaborator

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
Copy link
Collaborator

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

@julienw
Copy link
Collaborator Author

julienw commented May 25, 2021

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.

@pirquessa
Copy link
Collaborator

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 ?

@dunglas
Copy link
Owner

dunglas commented May 25, 2021

Ah oui ça devrait le faire aussi.

@julienw
Copy link
Collaborator Author

julienw commented May 25, 2021

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.

@Valdaria
Copy link

Valdaria commented May 25, 2021

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

@dunglas
Copy link
Owner

dunglas commented May 25, 2021

"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 ?

@julienw
Copy link
Collaborator Author

julienw commented May 25, 2021

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

@pirquessa
Copy link
Collaborator

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

image

@Valdaria
Copy link

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

@pirquessa
Copy link
Collaborator

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

@Valdaria
Copy link

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

@pirquessa
Copy link
Collaborator

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

@julienw
Copy link
Collaborator Author

julienw commented May 26, 2021

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.

@pirquessa
Copy link
Collaborator

#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 :)

@julienw
Copy link
Collaborator Author

julienw commented May 26, 2021

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.

@pirquessa
Copy link
Collaborator

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 👍

@julienw
Copy link
Collaborator Author

julienw commented May 26, 2021

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.

@pirquessa
Copy link
Collaborator

Je viens de changer un peu l'algo dans #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.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants