if (acsByIndex.first && !acsByIndex.second) {
// Since we're passing the ACS by value, we need to compute the return URL,
// so we'll need the target resource for real.
- recoverRelayState(request.getApplication(), request, target, false);
+ recoverRelayState(request.getApplication(), request, request, target, false);
}
+ pair<bool,bool> flag;
option = request.getParameter("isPassive");
- isPassive = (option && (*option=='1' || *option=='t'));
+ if (option) {
+ isPassive = (*option=='1' || *option=='t');
+ }
+ else {
+ flag = getBool("isPassive");
+ isPassive = (flag.first && flag.second);
+ }
if (!isPassive) {
option = request.getParameter("forceAuthn");
- forceAuthn = (option && (*option=='1' || *option=='t'));
+ if (option) {
+ forceAuthn = (*option=='1' || *option=='t');
+ }
+ else {
+ flag = getBool("forceAuthn");
+ forceAuthn = (flag.first && flag.second);
+ }
}
- acClass.second = request.getParameter("authnContextClassRef");
- acClass.first = (acClass.second!=NULL);
- acComp.second = request.getParameter("authnContextComparison");
- acComp.first = (acComp.second!=NULL);
+ if (acClass.second = request.getParameter("authnContextClassRef"))
+ acClass.first = true;
+ else
+ acClass = getString("authnContextClassRef");
+
+ if (acComp.second = request.getParameter("authnContextComparison"))
+ acComp.first = true;
+ else
+ acComp = getString("authnContextComparison");
}
else {
// We're running as a "virtual handler" from within the filter.
const PropertySet* settings = request.getRequestSettings().first;
pair<bool,bool> flag = settings->getBool("isPassive");
+ if (!flag.first)
+ flag = getBool("isPassive");
isPassive = flag.first && flag.second;
if (!isPassive) {
flag = settings->getBool("forceAuthn");
+ if (!flag.first)
+ flag = getBool("forceAuthn");
forceAuthn = flag.first && flag.second;
}
acClass = settings->getString("authnContextClassRef");
+ if (!acClass.first)
+ acClass = getString("authnContextClassRef");
acComp = settings->getString("authnContextComparison");
+ if (!acComp.first)
+ acComp = getString("authnContextComparison");
}
if (ECP)
const IDPSSODescriptor* role = NULL;
const EndpointType* ep = NULL;
const MessageEncoder* encoder = NULL;
+
+ // We won't need this for ECP, but safety dictates we get the lock here.
+ MetadataProvider* m=app.getMetadataProvider();
+ Locker locker(m);
if (ECP) {
encoder = m_ecp;
}
else {
// Use metadata to locate the IdP's SSO service.
- MetadataProvider* m=app.getMetadataProvider();
- Locker locker(m);
MetadataProvider::Criteria mc(entityID, &IDPSSODescriptor::ELEMENT_QNAME, samlconstants::SAML20P_NS);
entity=m->getEntityDescriptor(mc);
if (!entity.first) {
- m_log.error("unable to locate metadata for provider (%s)", entityID);
+ m_log.warn("unable to locate metadata for provider (%s)", entityID);
throw MetadataException("Unable to locate metadata for identity provider ($entityID)", namedparams(1, "entityID", entityID));
}
else if (!entity.second) {
- m_log.error("unable to locate SAML 2.0 identity provider role for provider (%s)", entityID);
- return make_pair(false,0L);
+ m_log.warn("unable to locate SAML 2.0 identity provider role for provider (%s)", entityID);
+ if (getParent())
+ return make_pair(false,0L);
+ throw MetadataException("Unable to locate SAML 2.0 identity provider role for provider ($entityID)", namedparams(1, "entityID", entityID));
}
// Loop over the supportable outgoing bindings.
}
}
if (!ep || !encoder) {
- m_log.error("unable to locate compatible SSO service for provider (%s)", entityID);
- return make_pair(false,0L);
+ m_log.warn("unable to locate compatible SSO service for provider (%s)", entityID);
+ if (getParent())
+ return make_pair(false,0L);
+ throw MetadataException("Unable to locate compatible SSO service for provider ($entityID)", namedparams(1, "entityID", entityID));
}
}
if (!req->getIssuer()) {
Issuer* issuer = IssuerBuilder::buildIssuer();
req->setIssuer(issuer);
- issuer->setName(app.getXMLString("entityID").second);
+ issuer->setName(app.getRelyingParty(entity.first)->getXMLString("entityID").second);
}
if (!req->getNameIDPolicy()) {
NameIDPolicy* namepol = NameIDPolicyBuilder::buildNameIDPolicy();