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

Don't set CurlOpt.CAINFO when verify=False #183

Open
T-256 opened this issue Dec 18, 2023 · 2 comments
Open

Don't set CurlOpt.CAINFO when verify=False #183

T-256 opened this issue Dec 18, 2023 · 2 comments
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@T-256
Copy link
Contributor

T-256 commented Dec 18, 2023

No description provided.

@T-256 T-256 added the enhancement New feature or request label Dec 18, 2023
@yifeikong
Copy link
Owner

yifeikong commented Dec 18, 2023

Very reasonable, I like that. But does curl work with it?

@T-256
Copy link
Contributor Author

T-256 commented Dec 18, 2023

Yes, I can verify it works by removing it from reuqests/session.py and curl.py.

@yifeikong yifeikong added the good first issue Good for newcomers label Jan 1, 2024
@yifeikong yifeikong added this to the v0.6 milestone Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
Status: Todo
Development

No branches or pull requests

2 participants