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

[neohub] Avoid too frequent requests to hub #15743

Merged
merged 5 commits into from
Dec 10, 2023

Conversation

andrewfg
Copy link
Contributor

@andrewfg andrewfg commented Oct 12, 2023

Background

The NeoHub bridge thing sometimes goes offline due to systemTimeout. Discussions on the Heatmiser developer forum indicate that this error may be due to the binding making requests too fast.

https://dev.heatmiser.com/t/need-help-with-neohub-spuriously-changing-thermostat-settings/818/1

Solution

The solution is to pipeline the requests and throttle them.

Fixes #15498

Signed-off-by: Andrew Fiddian-Green software@whitebear.ch

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
@andrewfg andrewfg added the bug An unexpected problem or unintended behavior of an add-on label Oct 12, 2023
@andrewfg andrewfg self-assigned this Oct 12, 2023
@andrewfg andrewfg added the work in progress A PR that is not yet ready to be merged label Oct 12, 2023
Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
@lolodomo
Copy link
Contributor

lolodomo commented Nov 3, 2023

@andrewfg : still really work in progress ?

@andrewfg
Copy link
Contributor Author

andrewfg commented Nov 3, 2023

still really work in progress ?

@lolodomo yes, unfortunately; I have been running it 7/24 and the PR seems not to solve the issue.

@andrewfg
Copy link
Contributor Author

andrewfg commented Dec 5, 2023

@lolodomo this one is good to go now :)

@andrewfg andrewfg requested a review from lolodomo December 5, 2023 10:33
@andrewfg andrewfg removed the work in progress A PR that is not yet ready to be merged label Dec 5, 2023
Copy link
Contributor

@lolodomo lolodomo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thank you

@lolodomo lolodomo merged commit 7da863a into openhab:main Dec 10, 2023
3 checks passed
@lolodomo lolodomo added this to the 4.1 milestone Dec 10, 2023
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
* [neohub] throttle requests to hub
* [neohub] handle websocket error; and attempt restart

---------

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Signed-off-by: Jørgen Austvik <jaustvik@acm.org>
@andrewfg andrewfg deleted the neohub-synch branch August 25, 2024 12:35
joni1993 pushed a commit to joni1993/openhab-addons that referenced this pull request Oct 15, 2024
* [neohub] throttle requests to hub
* [neohub] handle websocket error; and attempt restart

---------

Signed-off-by: Andrew Fiddian-Green <software@whitebear.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug An unexpected problem or unintended behavior of an add-on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[neohub] systemTimeout config param not working
2 participants