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

How could I use MBedTLS for SSL? #420

Closed
SeanOMik opened this issue Jul 9, 2020 · 4 comments · Fixed by #714
Closed

How could I use MBedTLS for SSL? #420

SeanOMik opened this issue Jul 9, 2020 · 4 comments · Fixed by #714

Comments

@SeanOMik
Copy link

SeanOMik commented Jul 9, 2020

Is there a way I could use MBedTLS for curl? I tried to do this:

set(USE_WINSSL OFF)
set(USE_OPENSSL OFF)
set(CMAKE_USE_WINSSL OFF)
set(CMAKE_USE_OPENSSL OFF)
set(CMAKE_USE_MBEDTLS ON)

But when I create a get request, I get no response with this error: Cert verify failed: BADCERT_NOT_TRUSTED

Is there a way I could use MBedTLS with CPR?

@SeanOMik SeanOMik changed the title Use MBedTLS for SSL. How could I use MBedTLS for SSL? Jul 9, 2020
@COM8 COM8 added this to the CPR 1.6.0 milestone Jul 9, 2020
@COM8
Copy link
Member

COM8 commented Jul 9, 2020

Currently I do not think we support MBedTLS, only OpenSSL and WinSSL.
This should be possible to add through.

@SeanOMik
Copy link
Author

SeanOMik commented Jul 9, 2020

Okay I'll just use WinSSL. I'm also having issues with OpenSSL on Linux, but I'll create a separate issue for it.

@COM8 COM8 modified the milestones: CPR 1.6.0, CPR 1.7.0 Feb 26, 2021
@COM8
Copy link
Member

COM8 commented Feb 26, 2021

@KingKili do you have time to add MBedTLS support?

@KingKili KingKili modified the milestones: CPR 1.7.0, CPR 1.8.0 Nov 24, 2021
COM8 added a commit that referenced this issue Mar 20, 2022
@COM8 COM8 mentioned this issue Mar 20, 2022
@COM8
Copy link
Member

COM8 commented Mar 20, 2022

#714 adds a mew option to enable Mbed TLS.

@COM8 COM8 closed this as completed in #714 Mar 20, 2022
COM8 added a commit that referenced this issue Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants