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

[monopriceaudio] Fix reconnect error race condition #15536

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

mlobstein
Copy link
Contributor

Fix race condition that occurs during reconnect when the binding disconnects before the initial status query responses are received. The polling job will still determine when the amplifier is no longer responding and set the thing status appropriately.

Signed-off-by: Michael Lobstein <[email protected]>
@mlobstein mlobstein added the bug An unexpected problem or unintended behavior of an add-on label Sep 1, 2023
Copy link
Contributor

@jlaur jlaur left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@jlaur jlaur merged commit b9a0e6f into openhab:main Sep 1, 2023
2 checks passed
@jlaur jlaur added this to the 4.1 milestone Sep 1, 2023
@mlobstein mlobstein deleted the monoprice_connectError branch September 1, 2023 22:28
@mlobstein
Copy link
Contributor Author

@jlaur Can this be considered if another 4.0.x patch release is done?

jlaur pushed a commit that referenced this pull request Sep 12, 2023
@jlaur jlaur added the patch A PR that has been cherry-picked to a patch release branch label Sep 12, 2023
@jlaur
Copy link
Contributor

jlaur commented Sep 12, 2023

Can this be considered if another 4.0.x patch release is done?

It seems low-risk, so let's do that. I have cherry-picked the commit for being included in 4.0.4 when that happens.

querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 21, 2023
querdenker2k pushed a commit to querdenker2k/openhab-addons that referenced this pull request Oct 29, 2023
Signed-off-by: Michael Lobstein <[email protected]>
Signed-off-by: querdenker2k <[email protected]>
austvik pushed a commit to austvik/openhab-addons that referenced this pull request Mar 27, 2024
Signed-off-by: Michael Lobstein <[email protected]>
Signed-off-by: Jørgen Austvik <[email protected]>
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 patch A PR that has been cherry-picked to a patch release branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants