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

Map ESP-IDF's mbedTLS to rust-native-tls #22

Open
ivmarkov opened this issue Nov 8, 2021 · 6 comments
Open

Map ESP-IDF's mbedTLS to rust-native-tls #22

ivmarkov opened this issue Nov 8, 2021 · 6 comments
Labels
help wanted Extra attention is needed rust-std

Comments

@ivmarkov
Copy link
Collaborator

ivmarkov commented Nov 8, 2021

... or alternatively, map ESP-IDF's partial OpenSSL implementation to rust-openssl.

Both solutions would require a lot of work and will likely result in patches against one of these upstream crates.

@igrr
Copy link

igrr commented Nov 8, 2021

Just a note, esp-idf will drop support for OpenSSL wrapper in 5.0 release. If it is necessary for rust, there is an option to extract the wrapper into a separate component, and maintain it as part for esp-rs organization.

@MabezDev
Copy link
Member

MabezDev commented Nov 22, 2021

I think our best bet is to add support for mbedtls to rust-native-tls. There is rust-mbedtls, however we probably need to PR some changes so that it it's possible to link against the mbedtls component in esp-idf instead of building from source.

I'll look into building rust-mbedtls against esp-idf, and I'll also open an issue upstream in rust-native-tls to make sure they are open to the implementation.

@georgik georgik moved this to In Progress in esp-rs Feb 8, 2022
@georgik georgik added the help wanted Extra attention is needed label May 17, 2022
@bsodmike
Copy link
Contributor

bsodmike commented Nov 4, 2022

Bump...

@MabezDev
Copy link
Member

MabezDev commented Nov 4, 2022

See sfackler/rust-native-tls#211 (comment) for an update

@bsodmike
Copy link
Contributor

bsodmike commented Nov 4, 2022

Thanks @MabezDev

@MabezDev MabezDev moved this from In Progress to Todo in esp-rs Feb 21, 2024
@MabezDev MabezDev removed their assignment Feb 21, 2024
@juliankrieger
Copy link

Is this issue still open? Brian Smith seems to have added support for little endian architectures in Ring a couple of weeks ago. See more here: https://www.reddit.com/r/rust/comments/1d7zzb8/which_tls_crate_do_you_recommend_to_use_for_an/

It seems like this commit (briansmith/ring@ed5b2a8) is not yet included in a release and I've not yet tried it, but it seems like ring should then work on esp32 systems?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed rust-std
Projects
Status: Todo
Development

No branches or pull requests

6 participants