Fix to default to taking the RACS URL from the application configuration.
Project: http://git-wip-us.apache.org/repos/asf/cxf-fediz/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-fediz/commit/2db18cef
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/2db18cef
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/2db18cef
Branch: refs/heads/master
Commit: 2db18ceffdd1c6547e6a589d7ce3bd798eda5ed5
Parents: 4808a7b
Author: Colm O hEigeartaigh <coheigea@apache.org>
Authored: Wed Aug 9 15:25:45 2017 +0100
Committer: Colm O hEigeartaigh <coheigea@apache.org>
Committed: Wed Aug 9 15:25:45 2017 +0100
----------------------------------------------------------------------
.../fediz/service/idp/beans/EndpointAddressValidator.java | 4 ++--
.../service/idp/beans/samlsso/AuthnRequestParser.java | 10 ++++++++++
2 files changed, 12 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/2db18cef/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
----------------------------------------------------------------------
diff --git a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
index de193b8..6a19554 100644
--- a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
+++ b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
@@ -46,7 +46,7 @@ public class EndpointAddressValidator {
Idp idpConfig = (Idp) WebUtils.getAttributeFromFlowScope(context, "idpConfig");
Application serviceConfig = idpConfig.findApplication(realm);
if (serviceConfig == null) {
- LOG.warn("No service config found for " + realm);
+ LOG.warn("No service config found for {}", realm);
return false;
}
@@ -66,7 +66,7 @@ public class EndpointAddressValidator {
Application serviceConfig = idpConfig.findApplication(realm);
if (serviceConfig == null) {
- LOG.warn("No service config found for " + realm);
+ LOG.warn("No service config found for {}", realm);
return false;
}
http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/2db18cef/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/samlsso/AuthnRequestParser.java
----------------------------------------------------------------------
diff --git a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/samlsso/AuthnRequestParser.java
b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/samlsso/AuthnRequestParser.java
index 3110eb1..92d0d7a 100644
--- a/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/samlsso/AuthnRequestParser.java
+++ b/services/idp-core/src/main/java/org/apache/cxf/fediz/service/idp/beans/samlsso/AuthnRequestParser.java
@@ -131,6 +131,16 @@ public class AuthnRequestParser {
}
LOG.debug("No AuthnRequest available to be parsed");
+
+ Idp idpConfig = (Idp) WebUtils.getAttributeFromFlowScope(context, "idpConfig");
+ String realm = retrieveRealm(context);
+ Application serviceConfig = idpConfig.findApplication(realm);
+ if (serviceConfig != null) {
+ String racs = serviceConfig.getPassiveRequestorEndpoint();
+ LOG.debug("Attempting to use the configured passive requestor endpoint instead:
{}", racs);
+ return racs;
+ }
+
return null;
}
|