fix(idx): canRemediate nested object logic in GenericRemediator - OKT…

…A-495228
okta · May 3, 2022 · b76570d · b76570d
1 parent 73bc0e0
commit b76570d
Show file tree

Hide file tree

Showing 3 changed files with 310 additions and 53 deletions.
diff --git a/lib/idx/remediators/GenericRemediator/util.ts b/lib/idx/remediators/GenericRemediator/util.ts
@@ -41,21 +41,19 @@ export function unwrapFormValue(remediation): Input {
 export function hasValidInputValue(input, values) {
   const fn = (input, values, requiredTracker) => {
     const { name, value, required } = input;
-    const isRequired = required 
-      || requiredTracker 
-      // TODO: confirm with backend why `required` meta is missing for authenticator remediation
-      || name === 'authenticator';
-    if (!isRequired) {
-      return true;
-    }
+    const isRequired = required || requiredTracker;
 
     if (Array.isArray(value)) {
       return value.reduce((acc, item) => {
         return acc && fn(item, values[name], isRequired);
       }, true);
-    } else {
-      return !!(values && values[name]);
     }
+
+    if (!isRequired) {
+      return true;
+    }
+
+    return !!(values && values[name]);
   };
 
   return fn(input, values, false);

diff --git a/lib/idx/types/api.ts b/lib/idx/types/api.ts
@@ -45,6 +45,9 @@ export type Input = {
   secret?: boolean;
   required?: boolean;
   options?: IdxOption[];
+  relatesTo?: IdxAuthenticator;
+  mutable?: boolean;
+  visible?: boolean;
 }