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

Encrypted Flash option with Example #152

Open
wants to merge 11 commits into
base: feature/encrypted-flash
Choose a base branch
from

Conversation

fred0plays
Copy link

Hi,

we have created a working example that supports flashing encrypted firmware via checkboxes in the example.
This implementation has been tested on 2 different ESP devices (WROVER and WROOM).

Hope it helps.

@brianignacio5
Copy link
Collaborator

brianignacio5 commented Aug 15, 2024

Did you take a look at the pull request: #145

Also it seems you are adding esp32P4 which is already added in current 0.4.5 Could you rebase your PR branch to current master, check the previous PR regarding flash encryption and we can update your changes ?

Sort of duplicate ?

@fred0plays
Copy link
Author

Hi, thank you, I will take a look.

Also I wanted to ask you, are there any plans of pushing the encrypted-flash feature branch into main, so this functionality can be used via CDN? If yes, do you have any idea when, or what needs to be done in order to do so? Please let me know if I can be of any help, since this would greatly help us. @brianignacio5

@brianignacio5
Copy link
Collaborator

Hi @fred0plays ! thank you for the feedback.

We want to push #145 or encrypt flash functionality but so far I've not received review to the PR itself. If I can have others test it we could move forward to add it to main and next release. If you can have the time please take a look.

@rotempco
Copy link

What is the state of this? Is it abandoned - or is it due to be merged into main?

@brianignacio5
Copy link
Collaborator

Hi @rotempco Basically changes are in #145 to add flash encryption but I need people to test PR changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants