diff --git a/maxkey-protocols/maxkey-protocol-saml-2.0/src/main/java/org/maxkey/authz/saml20/provider/xml/AttributeStatementGenerator.java b/maxkey-protocols/maxkey-protocol-saml-2.0/src/main/java/org/maxkey/authz/saml20/provider/xml/AttributeStatementGenerator.java index 9a9de7e0..a183d5f7 100644 --- a/maxkey-protocols/maxkey-protocol-saml-2.0/src/main/java/org/maxkey/authz/saml20/provider/xml/AttributeStatementGenerator.java +++ b/maxkey-protocols/maxkey-protocol-saml-2.0/src/main/java/org/maxkey/authz/saml20/provider/xml/AttributeStatementGenerator.java @@ -49,6 +49,9 @@ public class AttributeStatementGenerator { private final XMLObjectBuilderFactory builderFactory = Configuration.getBuilderFactory(); + public static String COMMA = ","; + public static String COMMA_ISO8859_1 = ","; //, ->, + public AttributeStatement generateAttributeStatement(AppsSAML20Details saml20Details,ArrayList grantedAuthoritys) { return generateAttributeStatement(saml20Details, grantedAuthoritys,null); @@ -82,8 +85,16 @@ public class AttributeStatementGenerator { if(ConstsBoolean.isTrue(saml20Details.getIsExtendAttr()) && saml20Details.getExtendAttr() != null) { ExtraAttrs extraAttrs=new ExtraAttrs(saml20Details.getExtendAttr()); for(ExtraAttr extraAttr : extraAttrs.getExtraAttrs()) { - logger.debug("Attribute : "+extraAttr.getAttr()+" , Vale : "+extraAttr.getValue()+" , Type : "+extraAttr.getType()); - attributeStatement.getAttributes().add(builderAttribute(extraAttr.getAttr(),extraAttr.getValue(),extraAttr.getType())); + extraAttr.setValue(extraAttr.getValue().replaceAll(COMMA_ISO8859_1, COMMA)); + logger.debug("Attribute : {} , Vale : {} , Type : {}", + extraAttr.getAttr(),extraAttr.getValue(),extraAttr.getType()); + + attributeStatement.getAttributes().add(builderAttribute( + extraAttr.getAttr(), + extraAttr.getValue(), + extraAttr.getType() + ) + ); } }