From 48ae10b5fa277c0eec38236ee0328d4df771c15b Mon Sep 17 00:00:00 2001 From: jbernhardt Date: Fri, 25 Jul 2014 20:46:08 +0200 Subject: [PATCH 1/2] [CXF-5906] Fixing multi claim value support --- .../main/java/org/apache/cxf/sts/claims/ClaimsManager.java | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java index 5d28721dac1..26739c8fc99 100644 --- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java +++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java @@ -336,9 +336,8 @@ protected List parseClaimsInAssertion(org.opensaml.saml1.core.As LOG.finest(" [" + value + "]"); } c.addValue(value); - collection.add(c); - break; } + collection.add(c); } } return collection; @@ -375,9 +374,8 @@ protected List parseClaimsInAssertion(org.opensaml.saml2.core.As LOG.finest(" [" + value + "]"); } c.addValue(value); - collection.add(c); - break; } + collection.add(c); } } return collection; From 24189840a3535cda1b5c11f03f9f2de165c11177 Mon Sep 17 00:00:00 2001 From: Jan Bernhardt Date: Fri, 25 Jul 2014 21:39:39 +0200 Subject: [PATCH 2/2] [CXF-5907] Removing disturbing claims validation check in case of federated claim mapping --- .../src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java | 1 - 1 file changed, 1 deletion(-) diff --git a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java index 26739c8fc99..dec567955a0 100644 --- a/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java +++ b/services/sts/sts-core/src/main/java/org/apache/cxf/sts/claims/ClaimsManager.java @@ -259,7 +259,6 @@ public ProcessedClaimCollection retrieveClaimValues(ClaimCollection claims, Clai ProcessedClaimCollection targetClaims = claimsMapper.mapClaims(relationship.getSourceRealm(), sourceClaims, relationship.getTargetRealm(), parameters); - validateClaimValues(claims, targetClaims); return targetClaims; }