X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Fclient%2Fsfaserverproxy.py;h=ca62ddf0e07991dadd61f41deeffe532f33c1103;hb=a0ef6e7c91c3c8fa376943d28fec8c5c204a78cd;hp=615a8b63f7ae16b63784776112fa7ea530697045;hpb=7aaa717b25b26927ecde5dc067e0c417ffdedec8;p=sfa.git diff --git a/sfa/client/sfaserverproxy.py b/sfa/client/sfaserverproxy.py index 615a8b63..ca62ddf0 100644 --- a/sfa/client/sfaserverproxy.py +++ b/sfa/client/sfaserverproxy.py @@ -2,7 +2,8 @@ # starting with 2.7.9 we need to turn off server verification import ssl -ssl_needs_unverified_context = hasattr(ssl, '_create_unverified_context') +try: turn_off_server_verify = { 'context' : ssl._create_unverified_context() } +except: turn_off_server_verify = {} import xmlrpclib from httplib import HTTPS, HTTPSConnection @@ -26,7 +27,7 @@ class ExceptionUnmarshaller(xmlrpclib.Unmarshaller): def close(self): try: return xmlrpclib.Unmarshaller.close(self) - except xmlrpclib.Fault, e: + except xmlrpclib.Fault as e: raise ServerException(e.faultString) ## @@ -48,13 +49,9 @@ class XMLRPCTransport(xmlrpclib.Transport): # create a HTTPS connection object from a host descriptor # host may be a string, or a (host, x509-dict) tuple host, extra_headers, x509 = self.get_host_info(host) - if not ssl_needs_unverified_context: - conn = HTTPSConnection(host, None, key_file = self.key_file, - cert_file = self.cert_file) - else: - conn = HTTPSConnection(host, None, key_file = self.key_file, - cert_file = self.cert_file, - context = ssl._create_unverified_context()) + conn = HTTPSConnection(host, None, key_file = self.key_file, + cert_file = self.cert_file, + **turn_off_server_verify) # Some logic to deal with timeouts. It appears that some (or all) versions # of python don't set the timeout after the socket is created. We'll do it @@ -85,13 +82,9 @@ class XMLRPCServerProxy(xmlrpclib.ServerProxy): # remember url for GetVersion # xxx not sure this is still needed as SfaServerProxy has this too self.url=url - if not ssl_needs_unverified_context: - xmlrpclib.ServerProxy.__init__(self, url, transport, allow_none=allow_none, - verbose=verbose) - else: - xmlrpclib.ServerProxy.__init__(self, url, transport, allow_none=allow_none, - verbose=verbose, - context=ssl._create_unverified_context()) + xmlrpclib.ServerProxy.__init__(self, url, transport, allow_none=allow_none, + verbose=verbose, + **turn_off_server_verify) def __getattr__(self, attr): logger.debug ("xml-rpc %s method:%s" % (self.url, attr))