Try-Catch arround Pollen

BadWolf42 commented 4 years ago

Hello, Dans le cadre de mon nettoyage d'automne des logs, je vous propose l'amélioration suivante.

J'ai énormément d'erreurs du type : [2019-11-24 01:01:36][ERROR] : Erreur sur la fonction cronHourly du plugin : Echec de la requête HTTP : https://www.pollens.fr/risks/thea/counties/67 cURL error : Operation timed out after 0 milliseconds with 0 out of 0 bytes received [2019-11-24 02:01:36][ERROR] : Erreur sur la fonction cronHourly du plugin : Echec de la requête HTTP : https://www.pollens.fr/risks/thea/counties/67 cURL error : Connection timed out after 30000 milliseconds

Et aussi des timouts récurrents sur les cron hourly. Ce n'est pas nouveau, j'arrive à en retrouver depuis juin 2019 et même en 2018 si je regarde dans les archives.

Les erreurs viennent de vraisemblablement de pollens.fr qui, de temps en temps, ne répond rien ou ferme immédiatement la connexion. Du coup je propose 2 mesures :

ajouter des try-catch autour des appel com_http + exec et ajouter des messages de log directement dedans
réduire les valeurs des timeouts afin de laisser plus de temps aux autres jobs du cron en cas de non réponse.

Bon weekend et merci, Bad Wolf

lunarok commented 4 years ago

J'ai baissé les timeout et au lieu de faire du try dans la classe du plugin, j'ai regarder pour m'appuyer sur la classse com_http dnas le commit suivant : https://github.com/lunarok/jeedom_vigilancemeteo/commit/a36d4357201c2b4c820b5e45b7337bfb965cc1fa

BadWolf42 commented 4 years ago

Ok merci, je suis en phase avec ces choix. Maintenant j'attends juste que cette version arrive depuis le market.

lunarok / jeedom_vigilancemeteo

Try-Catch arround Pollen #53