-
Notifications
You must be signed in to change notification settings - Fork 2
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
Fix proposal for CLIENT_TIMEOUT value too low #11
Comments
If a timeout of 300 sec is needed to make it work.. then there is something seriously wrong.. Suggest to find real issue instead of this workaround |
Agree, however, as I said in my previous post, still getting my head around HA, and Python. This could simply be a matter of slow hardware, or something more complicated. I'd like to continue to migrate my Home Automation to HA first, before digging in deeper. Just wanted to share my findings with the community so others get rid of the incomfort frustration :) |
I agree: I think this is OK as a workaround, but should not become part of the library. |
Closing this PR as it is not going to be implemented. |
Please see below as reference for a fix in regards to Home Assistant Core issue #110140
I have been able to work around this until a fixture can be applied:
Workaround is to set timeout in the incomfortclient library to a sufficiently high value. Downside is this needs to be reapplied after every Home Assistant Core update.
docker exec command to achieve this is:
docker exec -it homeassistant sh "/bin/sed -i 's/CLIENT_TIMEOUT = 20/CLIENT_TIMEOUT = 300/g' /usr/local/lib/python3.12/site-packages/incomfortclient/__init__.py"
Alternatively one could create a shell command which can be triggered per the usual ways:
The text was updated successfully, but these errors were encountered: