RIOT
RIOT copied to clipboard
pkg/mbedtls: add TLS support for LWIP
Contribution description
- The client node connects to openssl server and sends an encrypted message after a handshake and key exchange.
- mbedtls package is added.
- lwip package is also modified.
Testing procedure
- Board used for the test: same5-xpro.
- A wifi shield is used to connect the board to wifi.
Issues/PRs references
-Depends on PR #15671
Please rebase.
While you are at it, please also prefix the commit messages with the relevant subsystem, so pkg/mbedtls: and pkg/lwip:
It should not be necessary to copy the mbedtls header files into pkg/mbedtls/include/
@mariemC thanks for cleaning up the configuration files. The implementation and its representation in menuconfig look good at first sight. I do, unfortunately, not have time for a full review + testing. Anyway, some thoughts on the rest of the PR:
- Extend the test README so one knows what to execute (shell commands?), what to expect, and maybe give some context which certificates are included in the folder and how they are incorporated.
- The name of the test folder is suboptimal. Usually, we prefix folders that relate to a package with
pkg_. Nowpkg_mbedtlsexists already. Maybe something likepkg_mbedtls_lwip? In general I was wondering if we should execute some more of the built in mbedtls tests in RIOT. Possible to extract some of the tests that you include there? - Most of the Kconfig symbols that you include are enabled by default. I think they shouldn't, but the particular test case should enable the relevant ones.