You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cxf.apache.org by co...@apache.org on 2017/04/13 12:43:55 UTC

[1/2] cxf-fediz git commit: Fixing tests

Repository: cxf-fediz
Updated Branches:
  refs/heads/1.3.x-fixes 5eba7a0c8 -> 6341ca762


Fixing tests


Project: http://git-wip-us.apache.org/repos/asf/cxf-fediz/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-fediz/commit/314791d7
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/314791d7
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/314791d7

Branch: refs/heads/1.3.x-fixes
Commit: 314791d751b5dcc243944254b63387276c35329c
Parents: 5eba7a0
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Thu Apr 13 13:43:07 2017 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Thu Apr 13 13:43:23 2017 +0100

----------------------------------------------------------------------
 .../src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/314791d7/systests/idp/src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java
----------------------------------------------------------------------
diff --git a/systests/idp/src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java b/systests/idp/src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java
index 10a6ce7..47434f4 100644
--- a/systests/idp/src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java
+++ b/systests/idp/src/test/java/org/apache/cxf/fediz/systests/idp/IdpTest.java
@@ -1006,7 +1006,7 @@ public class IdpTest {
         webClient.close();
 
         // 2. now we logout from IdP
-        String logoutWReply = "https://localhost:123456";
+        String logoutWReply = "https://localhost:12345/badlogout";
         String idpLogoutUrl = "https://localhost:" + getIdpHttpsPort() + "/fediz-idp/federation?wa="
             + FederationConstants.ACTION_SIGNOUT + "&wreply=" + logoutWReply
             + "&wtrealm=urn:org:apache:cxf:fediz:fedizhelloworld";
@@ -1162,7 +1162,7 @@ public class IdpTest {
         webClient.close();
 
         // 2. now we logout from IdP
-        String logoutWReply = "https://localhost:123456";
+        String logoutWReply = "https://localhost:12345/badlogout";
         String idpLogoutUrl = "https://localhost:" + getIdpHttpsPort() + "/fediz-idp/federation?wa="
             + FederationConstants.ACTION_SIGNOUT + "&wreply=" + logoutWReply
             + "&wtrealm=urn:org:apache:cxf:fediz:fedizhelloworld3";


[2/2] cxf-fediz git commit: Removing printlns

Posted by co...@apache.org.
Removing printlns


Project: http://git-wip-us.apache.org/repos/asf/cxf-fediz/repo
Commit: http://git-wip-us.apache.org/repos/asf/cxf-fediz/commit/6341ca76
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/6341ca76
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/6341ca76

Branch: refs/heads/1.3.x-fixes
Commit: 6341ca762ebe2f00d993d539b8947470626fbb31
Parents: 314791d
Author: Colm O hEigeartaigh <co...@apache.org>
Authored: Thu Apr 13 13:43:46 2017 +0100
Committer: Colm O hEigeartaigh <co...@apache.org>
Committed: Thu Apr 13 13:43:46 2017 +0100

----------------------------------------------------------------------
 .../cxf/fediz/service/idp/beans/EndpointAddressValidator.java    | 4 ----
 1 file changed, 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/6341ca76/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
----------------------------------------------------------------------
diff --git a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
index 536a2e6..83299cb 100644
--- a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
+++ b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/beans/EndpointAddressValidator.java
@@ -55,7 +55,6 @@ public class EndpointAddressValidator {
 
     public boolean isValidSignoutAddress(RequestContext context, String endpointAddress, String realm)
         throws Exception {
-        System.out.println("EA: " + endpointAddress + " " + realm);
         if (endpointAddress == null) {
             return true;
         }
@@ -99,7 +98,6 @@ public class EndpointAddressValidator {
     }
 
     private boolean validateSignoutEndpointAddress(Application serviceConfig, String endpointAddress) {
-        System.out.println("HERE");
         if (serviceConfig.getLogoutEndpoint() == null
             && serviceConfig.getCompiledLogoutEndpointConstraint() == null) {
             LOG.error("Either the 'logoutEndpoint' or the 'logoutEndpointConstraint' "
@@ -120,8 +118,6 @@ public class EndpointAddressValidator {
             }
         }
         
-        System.out.println("RET FALSE");
-
         return false;
     }