fix: implement workaround for some 500 series controllers replying with invalid callbacks in case of SUC route assignment failure #6370
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
@zwave-js-bot automerge |
zwave-js-bot
previously approved these changes
Oct 5, 2023
|
@zwave-js-bot automerge |
zwave-js-bot
approved these changes
Oct 5, 2023
AlCalzone
changed the title
zwave-js-bot
approved these changes
Oct 5, 2023
11 tasks
AlCalzone
added a commit
that referenced
this pull request
Oct 5, 2023
The `v12` release was supposed to increase reliability of Z-Wave JS, primarily by detecting situations where the controller was unable to transmit due to excessive RF noise or being unresponsive and automatically taking the necessary steps to recover. Instead, it uncovered bugs and erratical behavior in the 500 series firmwares, which triggered the automatic recovery in situations where it was not necessary. In the worst case, this would cause Z-Wave JS to end up in an infinite loop or restart over and over. This patch should fix and/or work around most (if not all) of these issues. Really sorry for the inconvenience! ### Bugfixes * Fixed an infinite loop caused by assuming the controller was temporarily unable to transmit when when sending a command results in the transmit status `Fail` (#6361) * Added a workaround to avoid a restart loop caused by 500 series controllers replying with invalid commands when assigning routes back to the controller (SUC) failed (#6370, #6372) * Automatically recovering an unresponsive controller by restarting it or Z-Wave JS in case of a missing callback is now only done for `SendData` commands. Previously some commands which were expecting a specific command to be received from a node could also trigger this, even if that command was not technically a command callback. (#6373) * Fixed an issue where rebuilding routes would throw an error because of calling the wrong method internally (#6362)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When assigning or deleting a SUC return route fails, some 500 series controllers reply with an invalid callback that triggers the "unresponsive controller" detection added in version 12, effectively causing a boot loop.
This PR adds a workaround that considers these non-matching callbacks a command failure, so they are no longer "missing".
fixes: #6363