X-Git-Url: http://www.project-moonshot.org/gitweb/?a=blobdiff_plain;ds=sidebyside;f=shibsp%2Fhandler%2Fimpl%2FWAYFSessionInitiator.cpp;h=160000d552808f32cbbf5a9f67b48b440d84125b;hb=c9adabf4ac03ccedc541385da3b2ce38fd43c45d;hp=b9cc9c9fa56d458caff0839e67c895284fe2d3f6;hpb=daff0eafb385c665facde813b1f45fabc56a04d4;p=shibboleth%2Fsp.git diff --git a/shibsp/handler/impl/WAYFSessionInitiator.cpp b/shibsp/handler/impl/WAYFSessionInitiator.cpp index b9cc9c9..160000d 100644 --- a/shibsp/handler/impl/WAYFSessionInitiator.cpp +++ b/shibsp/handler/impl/WAYFSessionInitiator.cpp @@ -135,18 +135,9 @@ pair WAYFSessionInitiator::run(SPRequest& request, string& entityID, } // Validate the ACS for use with this protocol. - pair ACSbinding = ACS ? ACS->getString("Binding") : pair(false,nullptr); - if (ACSbinding.first) { - pair compatibleBindings = getString("compatibleBindings"); - if (compatibleBindings.first && strstr(compatibleBindings.second, ACSbinding.second) == nullptr) { - m_log.error("configured or requested ACS has non-SAML 1.x binding"); - throw ConfigurationException("Configured or requested ACS has non-SAML 1.x binding ($1).", params(1, ACSbinding.second)); - } - else if (strcmp(ACSbinding.second, samlconstants::SAML1_PROFILE_BROWSER_POST) && - strcmp(ACSbinding.second, samlconstants::SAML1_PROFILE_BROWSER_ARTIFACT)) { - m_log.error("configured or requested ACS has non-SAML 1.x binding"); - throw ConfigurationException("Configured or requested ACS has non-SAML 1.x binding ($1).", params(1, ACSbinding.second)); - } + if (ACS && !XMLString::equals(samlconstants::SAML11_PROTOCOL_ENUM, ACS->getProtocolFamily())) { + m_log.error("configured or requested ACS has non-SAML 1.x binding"); + throw ConfigurationException("Configured or requested ACS has non-SAML 1.x binding ($1).", params(1, ACS->getString("Binding").second)); } if (!discoveryURL.first)