entrouvert
/
lasso
16
0
Fork 0
Code Issues Pull Requests Releases Activity

saml-2.0/provider: when looking for endpoints take a null role as meanning « take the first one »

This commit is contained in:
Benjamin Dauvergne 2013-10-11 16:35:03 +02:00
parent 0bb846a355
commit e50981372f
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lasso/saml-2.0/provider.c
View File

@ -808,7 +808,9 @@ lasso_saml20_provider_get_endpoint_url(LassoProvider *provider,
continue;
if (! endpoint_type->binding)
continue;
if (endpoint_type->role != role)
if (endpoint_type->role != role \
&& role != LASSO_PROVIDER_ROLE_ANY \
&& role != LASSO_PROVIDER_ROLE_NONE)
continue;
if (! lasso_strisequal(endpoint_type->kind, kind))
continue;
@ -851,11 +853,14 @@ lasso_saml20_provider_get_artifact_resolution_service_index(LassoProvider *provi
continue;
/* endpoints are already properly ordered to provide the default endpoint first, so
* we just need to return the first matching one */
if (endpoint_type->role == provider->role &&
lasso_strisequal(endpoint_type->kind,kind))
{
*index = endpoint_type->index;
return 0;
if (endpoint_type->role == provider->role || provider->role ==
LASSO_PROVIDER_ROLE_NONE || provider->role ==
LASSO_PROVIDER_ROLE_ANY) {
if (lasso_strisequal(endpoint_type->kind,kind))
{
*index = endpoint_type->index;
return 0;
}
}
}
return -1;