Skip to content

Rollback with Arduino as ESP-IDF Component #5871

Closed
@GKFernandes

Description

@GKFernandes

Hello everyone,

I ran into the same issue as stated in: #3318 and also in #3749

I think this issue about the weakly defined verify0ta() should be fixed and made in such a way that the user can perform the verification later in the code instead of inside the initArduino() function.

Activity

deleted a comment from stale on Apr 20, 2022
self-assigned this
on May 13, 2022
me-no-dev

me-no-dev commented on May 13, 2022

@me-no-dev
Member

How about we add a new weak function bool verifyRollbackLater() to allow you to skip verification in initArduino and do it yourself later?

added this to the 2.0.4 milestone on May 13, 2022

1 remaining item

GKFernandes

GKFernandes commented on May 19, 2022

@GKFernandes
Author

Hello @me-no-dev , I think this could be a solution, but can the function verifyOta() just be removed?

What do you think @atanisoft @marcovannoord?

marcovannoord

marcovannoord commented on May 19, 2022

@marcovannoord

Yeah that would be a perfect solution i think, and it would just skip the verifyOTA() function.
In that case, if users want to use the verifyOTA to immediately verify rollback, then they can still use that function if they override the weak function.

me-no-dev

me-no-dev commented on May 19, 2022

@me-no-dev
Member

if you overwrite verifyRollbackLater, then verifyOTA will not be called at all. This whole block will not execute: https://github.com/espressif/arduino-esp32/blob/master/cores/esp32/esp32-hal-misc.c#L215-L226

marcovannoord

marcovannoord commented on May 19, 2022

@marcovannoord

Yeah i think that's the perfect solution. That way, users can add this feature, while keeping legacy things intact.

Repository owner moved this from In Progress to Done in Arduino ESP32 Core Project Roadmapon May 19, 2022
me-no-dev

me-no-dev commented on May 19, 2022

@me-no-dev
Member

Done :)

zekageri

zekageri commented on Nov 1, 2022

@zekageri

Any example for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Relationships

None yet

    Development

    Participants

    @marcovannoord@me-no-dev@zekageri@VojtechBartoska@GKFernandes

    Issue actions

      Rollback with Arduino as ESP-IDF Component · Issue #5871 · espressif/arduino-esp32