Merge pull request #66 from open-craft/edx1-sls-optional

Fix creation of metadata with no SLS, when using settings.get_sp_metadata()

Author: pitbulk

src/onelogin/saml2/metadata.py

@@ -74,7 +74,7 @@ def builder(sp, authnsign=False, wsign=False, valid_until=None, cache_duration=N
             organization = {}
 
         sls = ''
-        if 'singleLogoutService' in sp:
+        if 'singleLogoutService' in sp and 'url' in sp['singleLogoutService']:
             sls = """        <md:SingleLogoutService Binding="%(binding)s"
                                 Location="%(location)s" />\n""" % \
                 {

tests/src/OneLogin/saml2_tests/metadata_test.py

@@ -64,7 +64,7 @@ def testBuilder(self):
 
         security['authnRequestsSigned'] = True
         security['wantAssertionsSigned'] = True
-        del sp_data['singleLogoutService']
+        del sp_data['singleLogoutService']['url']
 
         metadata2 = OneLogin_Saml2_Metadata.builder(
             sp_data, security['authnRequestsSigned'],