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.

Sign up for GitHub

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

Jump to bottom

Unified Sequence Number(s): McuManager now tracks Sequence Numbers and rotates them for every single command/write/send #78

Merged
merged 1 commit into from
Sep 22, 2022
Merged

Unified Sequence Number(s): McuManager now tracks Sequence Numbers and rotates them for every single command/write/send #78

merged 1 commit into from
Sep 22, 2022

Commits on Sep 21, 2022

  1. Unified Sequence Number(s): McuManager now tracks Sequence Numbers an… 

    …d rotates them for every single command/write/send

Previously, only ImageManager was handling sequence numbers, and that too, only for upload. All other issued commands / writes defaulted to sequence number zero. I think a possible cause for issues in the validated / timeout stage was that when issuing commands that were not the upload, we were going back to sequence number zero. And they might've been dropped / confused by the rest of the code. Now they're integrated within McuManager, so they're used for all commands. This required deeper changes into the ImageManager logic, but I think we might've solved a couple of previous issues.

Maybe.
    @dinesharjani
    dinesharjani committed Sep 21, 2022
    Configuration menu
    Copy the full SHA
    49235ff View commit details
    Browse the repository at this point in the history