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.

Sign up for GitHub

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

Jump to bottom

Add missing SSL Options #467

Merged
merged 2 commits into from
Nov 7, 2022
Merged

Add missing SSL Options #467

merged 2 commits into from
Nov 7, 2022

Conversation

ETKNeil
Copy link
Contributor

@ETKNeil ETKNeil commented Sep 28, 2022

Add the missing curl options that can be found here : https://curl.se/libcurl/c/CURLOPT_SSL_OPTIONS.html

@ETKNeil ETKNeil force-pushed the add-curl-ssl-options branch from 3c86165 to 2a7cb9f Compare September 28, 2022 15:01
@alexcrichton
Copy link
Owner

I think systest/build.rs needs to be updated to indicate which version these constants were added in to appease CI

@ETKNeil ETKNeil force-pushed the add-curl-ssl-options branch from 2a7cb9f to 31eeba7 Compare November 7, 2022 08:55
@ETKNeil
Copy link
Contributor Author

ETKNeil commented Nov 7, 2022

I think systest/build.rs needs to be updated to indicate which version these constants were added in to appease CI

Hi I added the constants to systest as well as a useful link to the version. I hope this can be merged.
Please tell me if anything is missing

@alexcrichton alexcrichton merged commit a4958d1 into alexcrichton:main Nov 7, 2022
@lischach lischach mentioned this pull request Jun 13, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alexcrichton alexcrichton alexcrichton approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ETKNeil @alexcrichton