]> granicus.if.org Git - python/commitdiff
Fix Issue 9991: xmlrpc client ssl check faulty
authorSenthil Kumaran <orsenthil@gmail.com>
Thu, 18 Nov 2010 15:00:53 +0000 (15:00 +0000)
committerSenthil Kumaran <orsenthil@gmail.com>
Thu, 18 Nov 2010 15:00:53 +0000 (15:00 +0000)
Lib/test/test_xmlrpc.py
Lib/xmlrpc/client.py

index e082a2ba14a523b727d9ef493d272cdca13c02d5..62626b63955f5b575dbc4245fb8214112587575c 100644 (file)
@@ -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):
index d9c43c24b4137532c24d87e838b1a05643ca9926..b8559651e864075f1f37c8713be88d87cdd6e975 100644 (file)
@@ -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