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

Not clear instruction on REAME: check SPIFFS with root_ca.pem (https) and rsa_key.pem (signature check) #83

Closed
milkywade opened this issue Apr 10, 2022 · 2 comments · Fixed by #95

Comments

@milkywade
Copy link

This is a question / request for clarification

I am trying to make this example work: https://github.com/chrisjoyce911/esp32FOTA/blob/master/examples/HTTP/HTTPS.ino

In the example, it is not clear how to perform this step:
Step 3 : Provide SPIFFS filesystem with root_ca.pem of your webserver

I also checked the README.md - same cryptic instruction "For https or signature check: SPIFFS with root_ca.pem (https) and rsa_key.pem (signature check)"

How does actually one do that ?

Can the certificate check not be managed by a constant in the sketch as described in this article: https://www.lab4iot.com/2021/02/21/esp32-secure-firmware-update-over-the-air-ota/?unapproved=660&moderation-hash=f37b98b9077b522d86cd237ae1a442dc#comment-660

char *siteCertificate =
"-----BEGIN CERTIFICATE-----\n"
...

Many thanks in advance

@chrisjoyce911
Copy link
Owner

We are close to some changes around this , you may wish to try the changes,

#79

See example

https://github.com/thorrak/esp32FOTA/blob/c60cb7e4d76e22aae1003b52f46a26dfd8db70e8/examples/HTTP/HTTPS_without_root_cert.ino

@milkywade
Copy link
Author

milkywade commented Apr 25, 2022

thanks ! - I tried today to compile w the updated example, using v0.1.6, failed with "candidate expects 4 arguments, 5 provided", guess will have to wait until the new version has been committed

@tobozo tobozo mentioned this issue Sep 19, 2022
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 a pull request may close this issue.

2 participants