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

mbedTLS: add support for CURLOPT_CAINFO_BLOB

Closed
wants to merge 3 commits into from
Closed

Conversation

Floni
Copy link
Contributor

@Floni Floni commented Nov 29, 2021

No description provided.

@Floni
Copy link
Contributor Author

Floni commented Nov 30, 2021

Not sure about the failing tests, they seem unrelated to the PR. Could they be caused by server hiccups or something like that?

@bagder
Copy link
Member

bagder commented Nov 30, 2021

@Floni Right, those are (unfortunately) just our regular flaky builds and they're not failing because of your code.

bagder
bagder approved these changes Nov 30, 2021
docs/libcurl/opts/CURLOPT_CAINFO_BLOB.3 Outdated Show resolved Hide resolved
@Floni Floni requested a review from bagder Nov 30, 2021
bagder
bagder approved these changes Nov 30, 2021
@bagder
Copy link
Member

bagder commented Nov 30, 2021

Thanks!

@bagder bagder closed this in 8443f97 Nov 30, 2021
@bagder bagder reopened this Dec 1, 2021
@bagder
Copy link
Member

bagder commented Dec 1, 2021

Did you try test 678 with this code applied? It seems it fails now: https://curl.zuul.vexxhost.dev/build/a9732e2c883a4672aabd1c89dc4e40bd

The exact error output is:

 * Error importing ca cert blob  - mbedTLS: (-0x2180) X509 - The CRT/CRL/CSR format is invalid, e.g. different type expected

@bagder
Copy link
Member

bagder commented Dec 1, 2021

create an issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants