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 <[email protected]> Authored: Mon Feb 1 14:27:37 2016 +0000 Committer: Colm O hEigeartaigh <[email protected]> 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); } }
