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:50:41 GMT
Repository: cxf-fediz
Updated Branches:
  refs/heads/master eff7b768f -> a686f8333


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/315bf2e3
Tree: http://git-wip-us.apache.org/repos/asf/cxf-fediz/tree/315bf2e3
Diff: http://git-wip-us.apache.org/repos/asf/cxf-fediz/diff/315bf2e3

Branch: refs/heads/master
Commit: 315bf2e3cfd1edae3ec78144b127cfabb4c2fbdb
Parents: eff7b76
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:27:37 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/315bf2e3/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 a9448cc..be2333c 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