change(esp-tls): Improve support for wolfssl (IDFGH-15505)#16145
change(esp-tls): Improve support for wolfssl (IDFGH-15505)#16145gojimmypi wants to merge 1 commit into
Conversation
👋 Hello gojimmypi, we appreciate your contribution to this project! 📘 Please review the project's Contributions Guide for key guidelines on code, documentation, testing, and more. 🖊️ Please also make sure you have read and signed the Contributor License Agreement for this project. Click to see more instructions ...
Review and merge process you can expect ...
|
9b1a398 to
5697a4c
Compare
|
gojimmypi seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account. You have signed the CLA already but the status is still pending? Let us recheck it. |
|
Hi @gojimmypi, just migrated this PR to esp-wolfssl (check this). |
|
Hi @hrushikesh430 - yes, I see that there's now a submodule in the esp-wolfssl repo. That will likely pull in something like a half gig of mostly unrelated code, no? It is unfortunate that serious security continues to be considered an add-on, rather than a critical built-in, with wolfSSL integrated security removed from the EDP-IDF v6. (not even mentioned as a breaking change in the press release nor releases) Have you tested the esp-wolfssl with things like wolfSSH, wolfMQTT, wolfTPM, etc? Can I assume there's no longer interest in the Espressif wolfSSL Managed Components? I don't see how venturing off on customizations like that in espressif/esp-wolfssl#29 and the esp-wolfssl in general are helpful to serious industrial and commercial end users. I appreciate you asking my input. I just don't think that is the right direction to go. |
Could please take a look, sorry for previous misdirected comment. |
|
No worries. See my follow-up: #17683 (comment) response. |
Description
Improves support for wolfSSL in the ESP-IDF
esp-tlscomponent.I realize that
v5.5is already in beta1 as of a few days ago. I'm hopeful that this update can be included.Works best with wolfSSL v5.7.6 or later; v5.7.8 is better, more polish coming to
masterbranch.The wolfssl source code can also be set via
WOLFSSL_ROOTvia environment variable, cmake variable, oridf.py menuconfig.See ESP Registry for wolfssl and other components in the wolfssl namespace.
For getting started, see:
Related
Recent
esp-tlsspecific changes to wolfSSL:Overview:
Also:
Testing
Tested with my slightly revised esp_http_client_example.
Note the problem references in:
Checklist
Before submitting a Pull Request, please ensure the following: