Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add timeout to data providers #18

Closed
pskowronek opened this issue Oct 7, 2023 · 5 comments
Closed

Add timeout to data providers #18

pskowronek opened this issue Oct 7, 2023 · 5 comments

Comments

@pskowronek
Copy link
Owner

pskowronek commented Oct 7, 2023

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:

pskowronek added a commit that referenced this issue Oct 7, 2023
@pskowronek
Copy link
Owner Author

Fix pushed

@pskowronek
Copy link
Owner Author

Reverting version 0.2.0 of meteoalertapi due to: rolfberkenbosch/meteoalert-api#32

@pskowronek
Copy link
Owner Author

Request :timeout doesn't work for name resolution: psf/requests#4790 :/

@pskowronek
Copy link
Owner Author

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
Copy link
Owner Author

pskowronek commented Oct 7, 2023

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

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

No branches or pull requests

1 participant