summaryrefslogtreecommitdiffstats
path: root/Lib
diff options
context:
space:
mode:
authorSenthil Kumaran <orsenthil@gmail.com>2010-11-18 15:00:53 (GMT)
committerSenthil Kumaran <orsenthil@gmail.com>2010-11-18 15:00:53 (GMT)
commit8ce1f1ff83bf2521fc0f98a45aeb5904c69a1286 (patch)
treeb2b52d32525f7fb97ff3208ee65431c5de68e16e /Lib
parentb10089edbac3ab301f5b974cd7bb6a878bbb0bf0 (diff)
downloadcpython-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.py25
-rw-r--r--Lib/xmlrpc/client.py2
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