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

[deconz] refactoring and fix connection issues #9614

Merged
merged 4 commits into from
Dec 31, 2020
Merged

Conversation

J-N-K
Copy link
Member

@J-N-K J-N-K commented Dec 31, 2020

Fixes #9604
Maybe related to #9517
Follow-Up to #9307

The root cause seems to be that the websocket-connection was restarted even if it was not closed before (onError does not necessarily close the session, but the code triggered a (delayed) restart of the connection). In some cases the onError was followed by ?onClose which triggered an (immediate) restart of the connection, running two concurrent restart-attempts.

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
@J-N-K J-N-K added the bug An unexpected problem or unintended behavior of an add-on label Dec 31, 2020
Copy link
Member

@fwolter fwolter left a comment

Choose a reason for hiding this comment

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

LGTM

@fwolter fwolter merged commit d2bd511 into openhab:main Dec 31, 2020
@fwolter fwolter added this to the 3.1 milestone Dec 31, 2020
@J-N-K J-N-K deleted the deconz2 branch January 1, 2021 10:13
thinkingstone pushed a commit to thinkingstone/openhab-addons that referenced this pull request Nov 7, 2021
* fix connection attempts when thing disposed

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* refactor session handling

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* rename

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* refactoring

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
marcfischerboschio pushed a commit to bosch-io/openhab-addons that referenced this pull request May 5, 2022
* fix connection attempts when thing disposed

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* refactor session handling

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* rename

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>

* refactoring

Signed-off-by: Jan N. Klug <jan.n.klug@rub.de>
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.

[deconz] multiple buttonevents (HUE dimmer-switch, Ikea 5-button & Aquara Cube)
2 participants