Add timeout to data providers - Githubissues

pskowronek / epaper-clock-and-more

e-paper clock + weather + AQI + traffic delays - using Waveshare 2.7inch & 4.2inch eink displays running on Raspberry Pi

MIT License

38 stars 6 forks source link

Add timeout to data providers #18

Closed pskowronek closed 9 months ago

pskowronek commented 9 months ago

Due to how data from providers are being fetched it may happen when there's no internet connection that EPaper service will be killed by watchdog and wrong time will be displayed constantly on display.

Quick workaround is to add time outs (2s for connection, 4s for read) + newer version of meteoalertapi.

Ideally it would be nice to:

implement timeouts this way: https://www.peterbe.com/plog/best-practice-with-retries-with-requests (where also retry count can be specified)
let clock refresh display without waiting for data

pskowronek commented 9 months ago

Fix pushed

pskowronek commented 9 months ago

Reverting version 0.2.0 of meteoalertapi due to: https://github.com/rolfberkenbosch/meteoalert-api/issues/32

pskowronek commented 9 months ago

Request :timeout doesn't work for name resolution: https://github.com/psf/requests/issues/4790 :/

pskowronek commented 9 months ago

Don't want to add more advanced things like function timeouts (see), I will just update readme.md with workaround how to set system-wide settings for name resolver timeout.

pskowronek commented 9 months ago

Added SIGABRT handler to display shutdown icon to let ppl know that displayed time is not relevant