cxf-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From cohei...@apache.org
Subject [1/2] cxf-fediz git commit: Minor changes
Date Mon, 01 Feb 2016 14:51:07 GMT
Repository: cxf-fediz
Updated Branches:
  refs/heads/1.2.x-fixes 36c40d76e -> 8d86d6b6d


Minor changes


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

Branch: refs/heads/1.2.x-fixes
Commit: b680cf579b052cc0f43d27b61307704de286d94a
Parents: 36c40d7
Author: Colm O hEigeartaigh <coheigea@apache.org>
Authored: Mon Feb 1 14:27:37 2016 +0000
Committer: Colm O hEigeartaigh <coheigea@apache.org>
Committed: Mon Feb 1 14:50:55 2016 +0000

----------------------------------------------------------------------
 .../service/idp/protocols/TrustedIdpSAMLProtocolHandler.java    | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/cxf-fediz/blob/b680cf57/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/protocols/TrustedIdpSAMLProtocolHandler.java
----------------------------------------------------------------------
diff --git a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/protocols/TrustedIdpSAMLProtocolHandler.java
b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/protocols/TrustedIdpSAMLProtocolHandler.java
index 713dccb..e153b18 100644
--- a/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/protocols/TrustedIdpSAMLProtocolHandler.java
+++ b/services/idp/src/main/java/org/apache/cxf/fediz/service/idp/protocols/TrustedIdpSAMLProtocolHandler.java
@@ -93,7 +93,8 @@ public class TrustedIdpSAMLProtocolHandler implements TrustedIdpProtocolHandler
     public static final String REQUIRE_KEYINFO = "require.keyinfo";
     
     /**
-     * Whether the assertions contained in the Response must be signed or not. The default
is "true".
+     * Whether the assertions contained in the Response must be signed or not (if the response
itself
+     * is not signed). The default is "true".
      */
     public static final String REQUIRE_SIGNED_ASSERTIONS = "require.signed.assertions";
     
@@ -214,7 +215,6 @@ public class TrustedIdpSAMLProtocolHandler implements TrustedIdpProtocolHandler
                 new SecurityToken(id, validatorResponse.getCreated(), validatorResponse.getSessionNotOnOrAfter());
 
             idpToken.setToken(validatorResponse.getAssertionElement());
-            
             String whr = (String) WebUtils.getAttributeFromFlowScope(context,
                                                                      FederationConstants.PARAM_HOME_REALM);
             LOG.info("[IDP_TOKEN={}] created from [RP_TOKEN={}] issued by home realm [{}]",
@@ -401,7 +401,6 @@ public class TrustedIdpSAMLProtocolHandler implements TrustedIdpProtocolHandler
             protocolValidator.validateSamlResponse(samlResponse, crypto, null);
         } catch (WSSecurityException ex) {
             LOG.debug(ex.getMessage(), ex);
-            ex.printStackTrace();
             throw ExceptionUtils.toBadRequestException(null, null);
         }
     }


Mime
View raw message