From feaebfa7a689f435fa27dd47c9ae445a71d67708 Mon Sep 17 00:00:00 2001
From: Ryan Mathis <rmathis@unicon.net>
Date: Fri, 10 Feb 2023 07:11:51 -0700
Subject: [PATCH] Fixed max length validation for shib configurations

---
 ui/src/app/admin/component/ConfigurationForm.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/ui/src/app/admin/component/ConfigurationForm.js b/ui/src/app/admin/component/ConfigurationForm.js
index 3c5b2a84e..9e2020dce 100644
--- a/ui/src/app/admin/component/ConfigurationForm.js
+++ b/ui/src/app/admin/component/ConfigurationForm.js
@@ -18,7 +18,7 @@ export function ConfigurationForm({ configurations, configuration = {}, loading,
 
     const [names, setNames] = React.useState([]);
     
-    const { control, register, getValues, watch, formState: { errors, isValid }, handleSubmit } = useForm({
+    const { control, register, getValues, watch, formState, handleSubmit } = useForm({
         defaultValues: {
             ...configuration
         },
@@ -34,6 +34,8 @@ export function ConfigurationForm({ configurations, configuration = {}, loading,
         }
     });
 
+    const { errors, isValid } = formState;
+
     const properties = useProperties();
 
     const addProperties = (props) => {
@@ -113,8 +115,10 @@ export function ConfigurationForm({ configurations, configuration = {}, loading,
                                 placeholder={translator('label.configuration-name-placeholder')}
                                 {...register(`name`, {
                                     required: true,
+                                    maxLength: 255,
                                     value: configuration.value || null,
                                     validate: {
+
                                         unique: v => v.trim() === configuration.name || !includes(names, v)
                                     }
                                 })} />
@@ -160,8 +164,10 @@ export function ConfigurationForm({ configurations, configuration = {}, loading,
                                                         <Form.Control
                                                             type={p.displayType === 'number' ? 'number' : 'text'}
                                                             placeholder="value"
+                                                            isInvalid={ errors?.properties && errors.properties[idx] }
                                                             {...register(`properties.${idx}.propertyValue`, {
                                                                 setValueAs: v => (p.displayType === 'number' ? parseInt(v) : v),
+                                                                maxLength: 255,
                                                             })} />
                                                     </FloatingLabel>
                                                 :