Return is a protected python word so it can not be used as a parameter name in a method definition.

These fixes are there to allow for the usage or return_url or return as parameter name.

src/saml2/client_base.py

@@ -707,12 +707,20 @@ class Base(Entity):
         :return: A URL
         """
         args = {"entityID": entity_id}
-        for key in ["return", "policy", "returnIDParam"]:
+        for key in ["policy", "returnIDParam"]:
             try:
                 args[key] = kwargs[key]
             except KeyError:
                 pass
 
+        try:
+            args["return"] = kwargs["return_url"]
+        except KeyError:
+            try:
+                args["return"] = kwargs["return"]
+            except KeyError:
+                pass
+
         if "isPassive" in kwargs:
             if kwargs["isPassive"]:
                 args["isPassive"] = "true"

src/saml2/discovery.py

@@ -62,9 +62,12 @@ class DiscoveryServer(Entity):
 
     # -------------------------------------------------------------------------
 
-    def create_discovery_service_response(self, return_url,
+    def create_discovery_service_response(self, return_url=None,
                                           returnIDParam="entityID",
-                                          entity_id=None):
+                                          entity_id=None, **kwargs):
+        if return_url is None:
+            return_url = kwargs["return"]
+            
         if entity_id:
             qp = urlencode({returnIDParam: entity_id})
 

tests/disco_conf.py

@@ -1,23 +1,24 @@
 from saml2.extension.idpdisc import BINDING_DISCO
 
 from pathutils import full_path
+from pathutils import xmlsec_path
 
 BASE = "http://localhost:8088"
 
 CONFIG = {
-    "entityid" : "%s/disco.xml" % BASE,
+    "entityid": "%s/disco.xml" % BASE,
-    "name" : "Rolands Discoserver",
+    "name": "Rolands Discoserver",
     "service": {
         "ds": {
-            "endpoints" : {
+            "endpoints": {
                 "disco_service": [
                     ("%s/disco" % BASE, BINDING_DISCO),
                 ]
             },
         },
     },
-    "debug" : 1,
+    "debug": 1,
-    "xmlsec_binary" : None,
+    "xmlsec_binary": xmlsec_path,
     "metadata": {
         "local": [full_path("servera.xml")],
     },

tests/servera_conf.py

@@ -8,6 +8,7 @@ from saml2.saml import NAMEID_FORMAT_TRANSIENT
 from saml2.saml import NAMEID_FORMAT_PERSISTENT
 
 from pathutils import full_path
+from pathutils import xmlsec_path
 
 BASE = "http://lingon.catalogix.se:8087"
 
@@ -49,7 +50,7 @@ CONFIG = {
     "key_file": full_path("test.key"),
     "cert_file": full_path("test.pem"),
     "ca_certs": full_path("cacerts.txt"),
-    "xmlsec_binary": None,
+    "xmlsec_binary": xmlsec_path,
     "metadata": {
         "local": [full_path("idp_all.xml"), full_path("vo_metadata.xml")],
     },

tests/test_69_discovery.py

@@ -5,14 +5,17 @@ from pathutils import dotname
 
 __author__ = 'rolandh'
 
-def _eq(l1,l2):
+
+def _eq(l1, l2):
     return set(l1) == set(l2)
 
+
 def test_verify():
     ds = DiscoveryServer(config_file=dotname("disco_conf"))
     assert ds
     assert ds.verify_sp_in_metadata("urn:mace:example.com:saml:roland:sp")
 
+
 def test_construct_0():
     sp = Saml2Client(config_file=dotname("servera_conf"))
     url = sp.create_discovery_service_request("http://example.com/saml/disco",
@@ -20,6 +23,8 @@ def test_construct_0():
 
     assert url == "http://example.com/saml/disco?entityID=https%3A%2F%2Fexample.com%2Fsaml%2Fsp.xml"
 
+
+
 def test_construct_1():
     sp = Saml2Client(config_file=dotname("servera_conf"))
     url = sp.create_discovery_service_request("http://example.com/saml/disco",
@@ -27,21 +32,24 @@ def test_construct_1():
 
     assert url == "http://example.com/saml/disco?entityID=https%3A%2F%2Fexample.com%2Fsaml%2Fsp.xml"
 
+
 def test_construct_deconstruct_request():
     sp = Saml2Client(config_file=dotname("servera_conf"))
-    url = sp.create_discovery_service_request("http://example.com/saml/disco",
+    url = sp.create_discovery_service_request(
-                                              "https://example.com/saml/sp.xml",
+        "http://example.com/saml/disco",
-                                              is_passive=True,
+        "https://example.com/saml/sp.xml",
-                                              returnIDParam="foo",
+        is_passive=True,
-                                              return_url="https://example.com/saml/sp/disc")
+        returnIDParam="foo",
+        return_url="https://example.com/saml/sp/disc")
 
     print url
 
     ds = DiscoveryServer(config_file=dotname("disco_conf"))
     dsr = ds.parse_discovery_service_request(url)
     # policy is added by the parsing and verifying method
-    assert _eq(dsr.keys(),["return_url", "entityID", "returnIDParam",
+    assert _eq(dsr.keys(), ["return", "entityID", "returnIDParam",
-                           "isPassive", "policy"])
+                            "isPassive", "policy"])
+
 
 def test_construct_deconstruct_response():
     sp = Saml2Client(config_file=dotname("servera_conf"))
@@ -52,10 +60,14 @@ def test_construct_deconstruct_response():
                                               return_url="https://example.com/saml/sp/disc")
     ds = DiscoveryServer(config_file=dotname("disco_conf"))
     dsr = ds.parse_discovery_service_request(url)
-    args = dict([(key, dsr[key]) for key in ["returnIDParam", "return_url"]])
+    args = dict([(key, dsr[key]) for key in ["returnIDParam", "return"]])
     url = ds.create_discovery_service_response(
-                                  entity_id="https://example.com/saml/idp.xml",
+        entity_id="https://example.com/saml/idp.xml",
-                                  **args)
+        **args)
 
     idp_id = sp.parse_discovery_service_response(url, returnIDParam="foo")
     assert idp_id == "https://example.com/saml/idp.xml"
+
+
+if __name__ == "__main__":
+    test_construct_deconstruct_response()