From e059c9b0c246d53658d3db9c7afe01104ccea8e5 Mon Sep 17 00:00:00 2001 From: chasegawa Date: Mon, 28 Jun 2021 13:36:00 -0700 Subject: [PATCH] SHIBUI-1788 strip whitespace from "name" field --- .../admin/ui/domain/CustomEntityAttributeDefinition.java | 6 ++++++ .../admin/ui/domain/RelyingPartyOverrideProperty.java | 7 +++++++ 2 files changed, 13 insertions(+) diff --git a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/CustomEntityAttributeDefinition.java b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/CustomEntityAttributeDefinition.java index c2572e9d8..fdb9f9e83 100644 --- a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/CustomEntityAttributeDefinition.java +++ b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/CustomEntityAttributeDefinition.java @@ -12,6 +12,7 @@ import javax.persistence.JoinColumn; import javax.persistence.Transient; +import org.apache.commons.lang3.StringUtils; import org.hibernate.annotations.Fetch; import org.hibernate.annotations.FetchMode; import org.hibernate.envers.Audited; @@ -120,6 +121,11 @@ public void setDisplayType(String displayType) { // This is here to comply with the interface only and should not be used to change the value in this implementation } + @Override + public void setName(String name) { + this.name = StringUtils.strip(name); + } + public void updateExamplesList() { examples = customAttrListDefinitions; } diff --git a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/RelyingPartyOverrideProperty.java b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/RelyingPartyOverrideProperty.java index 4aa854e62..159501fab 100644 --- a/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/RelyingPartyOverrideProperty.java +++ b/backend/src/main/java/edu/internet2/tier/shibboleth/admin/ui/domain/RelyingPartyOverrideProperty.java @@ -2,6 +2,8 @@ import java.util.Set; +import org.apache.commons.lang3.StringUtils; + import lombok.Getter; import lombok.Setter; import lombok.ToString; @@ -50,4 +52,9 @@ public void setDefaultValues(Set defaults) { defaultValues = defaults; examples = defaults; } + + @Override + public void setName(String name) { + this.name = StringUtils.strip(name); + } } \ No newline at end of file