changeset: 92776:d9c52836aec8 branch: 2.7 user: Benjamin Peterson date: Fri Oct 03 17:27:05 2014 -0400 files: Lib/ssl.py Lib/test/test_ssl.py Misc/NEWS description: also use openssl envvars to find certs on windows (closes #22449) Patch by Christian Heimes and Alex Gaynor. diff -r 5ef28c22dc24 -r d9c52836aec8 Lib/ssl.py --- a/Lib/ssl.py Fri Oct 03 19:07:47 2014 +0200 +++ b/Lib/ssl.py Fri Oct 03 17:27:05 2014 -0400 @@ -376,8 +376,7 @@ if sys.platform == "win32": for storename in self._windows_cert_stores: self._load_windows_store_certs(storename, purpose) - else: - self.set_default_verify_paths() + self.set_default_verify_paths() def create_default_context(purpose=Purpose.SERVER_AUTH, cafile=None, diff -r 5ef28c22dc24 -r d9c52836aec8 Lib/test/test_ssl.py --- a/Lib/test/test_ssl.py Fri Oct 03 19:07:47 2014 +0200 +++ b/Lib/test/test_ssl.py Fri Oct 03 17:27:05 2014 -0400 @@ -1058,6 +1058,14 @@ self.assertRaises(TypeError, ctx.load_default_certs, None) self.assertRaises(TypeError, ctx.load_default_certs, 'SERVER_AUTH') + def test_load_default_certs_env(self): + ctx = ssl.SSLContext(ssl.PROTOCOL_TLSv1) + with support.EnvironmentVarGuard() as env: + env["SSL_CERT_DIR"] = CAPATH + env["SSL_CERT_FILE"] = CERTFILE + ctx.load_default_certs() + self.assertEqual(ctx.cert_store_stats(), {"crl": 0, "x509": 1, "x509_ca": 0}) + def test_create_default_context(self): ctx = ssl.create_default_context() self.assertEqual(ctx.protocol, ssl.PROTOCOL_SSLv23) diff -r 5ef28c22dc24 -r d9c52836aec8 Misc/NEWS --- a/Misc/NEWS Fri Oct 03 19:07:47 2014 +0200 +++ b/Misc/NEWS Fri Oct 03 17:27:05 2014 -0400 @@ -31,6 +31,9 @@ Library ------- +- Issue #22449: In the ssl.SSLContext.load_default_certs, consult the + enviromental variables SSL_CERT_DIR and SSL_CERT_FILE on Windows. + - Issue #8473: doctest.testfile now uses universal newline mode to read the test file.