diff options
| author | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-05-31 07:13:43 +0000 |
|---|---|---|
| committer | Benjamin Dauvergne <bdauvergne@entrouvert.com> | 2010-05-31 07:13:43 +0000 |
| commit | a0fb1580ad89ad3cbb7ef7bfb8db0f10c7d0d15f (patch) | |
| tree | f2563156ec3d70e059418eeb1efb671980a3f443 | |
| parent | 9d22f29e55524034dfda34c15b76f1b0b78c4413 (diff) | |
| download | lasso-a0fb1580ad89ad3cbb7ef7bfb8db0f10c7d0d15f.tar.gz lasso-a0fb1580ad89ad3cbb7ef7bfb8db0f10c7d0d15f.tar.xz lasso-a0fb1580ad89ad3cbb7ef7bfb8db0f10c7d0d15f.zip | |
SAMLv2: simplify lasso_saml20_provider_accept_http_method by only checking for remote provider support
* Whatever we do, with asyncrhonous bindings the remote provider can
return the response with any asynchronous binding.
| -rw-r--r-- | lasso/saml-2.0/provider.c | 15 |
1 files changed, 3 insertions, 12 deletions
diff --git a/lasso/saml-2.0/provider.c b/lasso/saml-2.0/provider.c index 60b588ff..58605aaf 100644 --- a/lasso/saml-2.0/provider.c +++ b/lasso/saml-2.0/provider.c @@ -697,18 +697,9 @@ lasso_saml20_provider_accept_http_method(LassoProvider *provider, LassoProvider protocol_profile = g_strdup_printf("%s %s", profile_names[protocol_type], http_methods[http_method+1]); - /* special hack for single sign on */ - if (protocol_type == LASSO_MD_PROTOCOL_TYPE_SINGLE_SIGN_ON) { - /* no need to check for the response, it uses another canal - * (AssertionConsumerService) */ - rc = (lasso_provider_get_metadata_list(remote_provider, protocol_profile) != NULL); - - } else { - if (lasso_provider_get_metadata_list(provider, protocol_profile) && - lasso_provider_get_metadata_list(remote_provider, protocol_profile)) { - rc = TRUE; - } - } + /* just check if remote provider can receive the request, remote provider will have to check + * how to return the response itself */ + rc = (lasso_provider_get_metadata_list(remote_provider, protocol_profile) != NULL); lasso_release_string(protocol_profile); return rc; } |