-
Notifications
You must be signed in to change notification settings - Fork 14
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
Updated wolfSSL to v5.7.2 #26
base: master
Are you sure you want to change the base?
Conversation
wolfSSL is also supported as a Managed Component: https://components.espressif.com/components/wolfssl/wolfssl There's an example of a flexible |
@frankencode check out my latest components/wolfssl/Kconfig that you may be interested in using. |
Thanks Jim! That's a lot of configure flags! I'll try to merge some from your user_settings/Kconfig when time allows. Another thing I wanted to do is to port the tests to pytest as used by other esp-idf components.. Furthermore I had to do some workarounds in the user_settings when removing "SINGLE_THREADED". You might want to check this out at the end of my user_settings. |
Hi @frankencode - that's interesting. I recall some time ago there was a problem with I've not personally encountered that recently. I'll take a look. Thanks for the heads up. In the meantime, you may also be interested in wolfSSL/wolfssl#7953; There I've moved some of the common config |
@frankencode I've not been able to reproduce any compiler errors related to the Other areas of interest: |
@gojimmypi Yes, I moved last week to latest ESP-IDF 5.4 and it compiles now without further issues when I comment out SINGLE_THREADED. A workaround in user_settings.h is not needed anymore. |
…P, no TLS13 and no SINGLE_THREADED
Checking for memory allocation failures is essential and should never be disabled. This increases the binary size only minimally. E.g. performance impact on the C6: * binary size increases by 14864 bytes * free heap is reduced by 8 bytes
Change summary: