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

Cert Bundle might also be a directory #748

Open
e7p opened this issue Mar 9, 2022 · 1 comment
Open

Cert Bundle might also be a directory #748

e7p opened this issue Mar 9, 2022 · 1 comment

Comments

@e7p
Copy link

e7p commented Mar 9, 2022

When using the requests API with the certificate parameter, it is also possible to enter e.g. /etc/ssl/certs as a certificate bundle and it will result in verifying against all certificates lying in this path.

Though, I would recommend also including os.path.isdir() in the statement.

if not os.path.isfile(cert_bundle):

@e7p
Copy link
Author

e7p commented Mar 9, 2022

Can be linked to #749

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

No branches or pull requests

1 participant