Merge pull request Uninett#56 from jdennis/acs-provider

olavmrk · olavmrk · commit 21f4acf27ef8 · 2015-11-05T16:59:57.000+01:00
Role maybe unknown when assertion consumer url is looked up
diff --git a/auth_mellon_util.c b/auth_mellon_util.c
@@ -1827,7 +1827,9 @@ char *am_get_assertion_consumer_service_by_binding(LassoProvider *provider, cons
     }
 
     if (selected_descriptor) {
-        url = lasso_provider_get_metadata_one(provider, selected_descriptor);
+        url = lasso_provider_get_metadata_one_for_role(provider,
+                                                       LASSO_PROVIDER_ROLE_SP,
+                                                       selected_descriptor);
     }
 
     lasso_release_list_of_strings(descriptors);

Original file line number	Diff line number	Diff line change
`@@ -1827,7 +1827,9 @@ char am_get_assertion_consumer_service_by_binding(LassoProvider provider, cons`
`1827`	`1827`	`}`
`1828`	`1828`
`1829`	`1829`	`if (selected_descriptor) {`
`1830`		`- url = lasso_provider_get_metadata_one(provider, selected_descriptor);`
	`1830`	`+ url = lasso_provider_get_metadata_one_for_role(provider,`
	`1831`	`+ LASSO_PROVIDER_ROLE_SP,`
	`1832`	`+ selected_descriptor);`
`1831`	`1833`	`}`
`1832`	`1834`
`1833`	`1835`	`lasso_release_list_of_strings(descriptors);`