-
Notifications
You must be signed in to change notification settings - Fork 4
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
More CA paths #1
Comments
Thanks for the pointer to the other paths! I'm not sure what you mean about path vs. file, though. Does httplib2 work if a directory name is used? |
Oh sorry, I don't know if httplib2 actually supports the feature. Since Python 3.2 the CA files can be loaded from a directory, too. The files in the directory must have a specific naming scheme so OpenSSL can map the cert's hash to a file name. http://docs.python.org/3.2/library/ssl.html#ssl.SSLContext.load_verify_locations |
OK, I'll verify that httplib2 supports that before including the directories. |
Fedora Core 19 has the certs in /etc/ssl/certs/ca-bundle.crt |
@jayvdb Not quite. Distros and OSes all have various (and usually more than one) locations for trust stores. The most-canonical location for RHEL, Fedora and Mageia (and derivatives and probably various smaller distros) is Really if you're trying to use the 'system' location you should not be specifying a location at all but letting OpenSSL handle it; see e.g. https://docs.python.org/2/library/ssl.html#ssl.SSLContext.load_default_certs . But as I read it, httplib2's design does not really allow for this, it is expecting the 'locater plugin' to provide a filesystem location. |
I would propose adopting the approach that go's stdlib does - https://golang.org/src/crypto/x509/root_linux.go |
Hi,
you can find more CA paths in my PEP draft at https://bitbucket.org/tiran/peps/src/5ac9cd851f24554621be1c475a1b8ab2a313208f/pep-9999.txt?at=default#cl-269
You should favor CApath over CAfile if the Python version supports it. A CApath is faster becaues it loads CAs on demand.
Christian
The text was updated successfully, but these errors were encountered: