openstack/deb-python-pysaml2
Code Issues Proposed changes

Avoid hard xmlsec_binary requirement for ECP client.

Browse Source 
By providing an already initialized Config() instance, an ECP
client using another CryptoBackend than xmlsec1 can be instantiated.
This commit is contained in:
Fredrik Thulin
2013-05-06 14:10:40 +02:00
parent 92cc3beccb
commit 42b916f382
2 changed files with 17 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
src/saml2/ecp_client.py
View File

@@ -48,21 +48,26 @@ class Client(Entity):
def __init__(self, user, passwd, sp="", idp=None, metadata_file=None, def __init__(self, user, passwd, sp="", idp=None, metadata_file=None,
xmlsec_binary=None, verbose=0, ca_certs="", xmlsec_binary=None, verbose=0, ca_certs="",
disable_ssl_certificate_validation=True, key_file=None, disable_ssl_certificate_validation=True, key_file=None,
cert_file=None): cert_file=None, config=None):
""" """
:param user: user name :param user: user name
:param passwd: user password :param passwd: user password
:param sp: The SP URL :param sp: The SP URL
:param idp: The IdP PAOS endpoint :param idp: The IdP PAOS endpoint
:param metadata_file: Where the metadata file is if used :param metadata_file: Where the metadata file is if used
:param xmlsec_binary: Where the xmlsec1 binary can be found :param xmlsec_binary: Where the xmlsec1 binary can be found (*)
:param verbose: Chatty or not :param verbose: Chatty or not
:param ca_certs: is the path of a file containing root CA certificates :param ca_certs: is the path of a file containing root CA certificates
for SSL server certificate validation. for SSL server certificate validation (*)
:param disable_ssl_certificate_validation: If :param disable_ssl_certificate_validation: If
disable_ssl_certificate_validation is true, SSL cert validation disable_ssl_certificate_validation is true, SSL cert validation
will not be performed. will not be performed (*)
:param key_file: Private key filename (*)
:param cert_file: Certificate filename (*)
:param config: Config() instance, overrides all the parameters marked
with an asterisk (*) above
""" """
if not config:
config = Config() config = Config()
config.disable_ssl_certificate_validation = \ config.disable_ssl_certificate_validation = \
disable_ssl_certificate_validation disable_ssl_certificate_validation

3
tests/test_63_ecp.py
View File

@@ -38,8 +38,7 @@ class DummyResponse(object):
def test_complete_flow(): def test_complete_flow():
client = ecp_client.Client("user", "password", client = ecp_client.Client("user", "password",
metadata_file=full_path("idp_all.xml"), metadata_file=full_path("idp_all.xml"))
xmlsec_binary=xmlsec_path)
sp = Saml2Client(config_file=dotname("servera_conf")) sp = Saml2Client(config_file=dotname("servera_conf"))
idp = Server(config_file=dotname("idp_all_conf")) idp = Server(config_file=dotname("idp_all_conf"))