[5/5] cxf-fediz git commit: Fix to default to taking the RACS URL from the application configuration.

2017-08-09 Thread coheigea
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/f71e6200
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/f71e6200
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/f71e6200

Branch: refs/heads/1.4.x-fixes
Commit: f71e62006bda7f83f113a52e90a16e613d0837ba
Parents: ea3124e
Author: Colm O hEigeartaigh 
Authored: Wed Aug 9 15:25:45 2017 +0100
Committer: Colm O hEigeartaigh 
Committed: Wed Aug 9 15:29:10 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/f71e6200/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/f71e6200/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;
 }
 



[5/5] cxf-fediz git commit: Fix to default to taking the RACS URL from the application configuration.

2017-08-09 Thread coheigea
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 
Authored: Wed Aug 9 15:25:45 2017 +0100
Committer: Colm O hEigeartaigh 
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;
 }