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

Suggestion: Asio is outdated, consider upgrading & add WolfSSL support? (IDFGH-2138) #4296

Closed
huming2207 opened this issue Nov 6, 2019 · 3 comments

Comments

@huming2207
Copy link
Contributor

Environment

(other irrelevant stuff removed)

Problem Description

Hi there,

I've posted this issue on the forum before but no one replied. So I think I might have posted that on the wrong place. Here's the original topic: https://www.esp32.com/viewtopic.php?f=13&t=12973

Basically, nowadays Asio supports WolfSSL as the SSL library backend (ref: https://www.wolfssl.com/wolfssl-support-asio-boost-asio-c-libraries/) since earlier this year. Meanwhile, ESP-IDF also has WolfSSL included. However, the Asio library in ESP-IDF hasn't been updated for a year and it does not have SSL/TLS support.

Could you guys please upgrade this library? Thanks in advance!

Regards,
Jackson

@github-actions github-actions bot changed the title Suggestion: Asio is outdated, consider upgrading & add WolfSSL support? Suggestion: Asio is outdated, consider upgrading & add WolfSSL support? (IDFGH-2138) Nov 6, 2019
@david-cermak
Copy link
Collaborator

Hi @huming2207

Thank you for this suggestion. Indeed the esp-asio hasn't been updated for a long time. Will update to the latest master soon.
As for the TLS support, this may take some time however, since ESP ASIO port is really basic with no SSL support and it's preferred to use esp-tls rather than WolfSSL directly. With latest ASIO it should still be possible to use WolfSSL setting define ASIO_USE_WOLFSSL on project level.

@huming2207
Copy link
Contributor Author

Hi @david-cermak ,

Thanks for your reply. I agree. If esp-tls can be used as a backed of Asio on ESP-IDF, it should be better.

Meanwhile, don't forget the WolfSSL has a dual-license model. It may cause some problems with some commercial projects.

Jackson.

@schmidma
Copy link
Contributor

schmidma commented Apr 1, 2020

Is there a new status regarding asio ssl support in esp-idf?

As I can see asio requests file system support from wolfSSL when using asio and wolfSSL with ASIO_USE_WOLFSSL.
E.g. asio::ssl::context::load_verify_file(...) requests usage of SSL_CTX_load_verify_locations(...) which is implemented by wolfSSL in wolfSSL_CTX_load_verify_locations(...) but disabled in the esp version with NO_FILESYSTEM.

Also when porting asio to use esp-tls there seems to be no asio-native option to disable file system related implementations.

Are there any plans to support the ssl set of asio?

gabsuren pushed a commit to gabsuren/esp-protocols-1 that referenced this issue May 28, 2022
gabsuren pushed a commit to gabsuren/esp-protocols-1 that referenced this issue Jun 3, 2022
gabsuren pushed a commit to gabsuren/esp-protocols-1 that referenced this issue Jun 22, 2022
euripedesrocha pushed a commit to euripedesrocha/esp-protocols that referenced this issue Oct 17, 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

No branches or pull requests

3 participants