diff options
author | Senthil Kumaran <orsenthil@gmail.com> | 2010-11-18 15:00:53 (GMT) |
---|---|---|
committer | Senthil Kumaran <orsenthil@gmail.com> | 2010-11-18 15:00:53 (GMT) |
commit | 8ce1f1ff83bf2521fc0f98a45aeb5904c69a1286 (patch) | |
tree | b2b52d32525f7fb97ff3208ee65431c5de68e16e /Lib | |
parent | b10089edbac3ab301f5b974cd7bb6a878bbb0bf0 (diff) | |
download | cpython-8ce1f1ff83bf2521fc0f98a45aeb5904c69a1286.zip cpython-8ce1f1ff83bf2521fc0f98a45aeb5904c69a1286.tar.gz cpython-8ce1f1ff83bf2521fc0f98a45aeb5904c69a1286.tar.bz2 |
Fix Issue 9991: xmlrpc client ssl check faulty
Diffstat (limited to 'Lib')
-rw-r--r-- | Lib/test/test_xmlrpc.py | 25 | ||||
-rw-r--r-- | Lib/xmlrpc/client.py | 2 |
2 files changed, 26 insertions, 1 deletions
diff --git a/Lib/test/test_xmlrpc.py b/Lib/test/test_xmlrpc.py index e082a2b..62626b6 100644 --- a/Lib/test/test_xmlrpc.py +++ b/Lib/test/test_xmlrpc.py @@ -149,6 +149,31 @@ class XMLRPCTestCase(unittest.TestCase): ('host.tld', [('Authorization', 'Basic dXNlcg==')], {})) + def test_ssl_presence(self): + #Check for ssl support + have_ssl = False + if hasattr(socket, 'ssl'): + have_ssl = True + else: + try: + import ssl + except: + pass + else: + have_ssl = True + try: + xmlrpc.client.ServerProxy('https://localhost:9999').bad_function() + except: + exc = sys.exc_info() + if exc[0] == socket.error: + self.assertTrue(have_ssl, + "No SSL support, but xmlrpclib reports supported.") + elif exc[0] == NotImplementedError and str(exc[1]) == \ + "your version of http.client doesn't support HTTPS": + self.assertFalse(have_ssl, + "SSL supported, but xmlrpclib reports not.") + else: + self.fail("Unable to determine status of SSL check.") class HelperTestCase(unittest.TestCase): diff --git a/Lib/xmlrpc/client.py b/Lib/xmlrpc/client.py index d9c43c2..b855965 100644 --- a/Lib/xmlrpc/client.py +++ b/Lib/xmlrpc/client.py @@ -1330,7 +1330,7 @@ class SafeTransport(Transport): if self._connection and host == self._connection[0]: return self._connection[1] - if not hasattr(socket, "ssl"): + if not hasattr(http.client, "ssl"): raise NotImplementedError( "your version of http.client doesn't support HTTPS") # create a HTTPS connection object from a host descriptor |