[Fleet] Allow empty strings for required text fields in package polic…

…ies (elastic#123610) * Allow empty strings for required text fields in package policies * make empty yaml check more explicit
yctercero · Jan 25, 2022 · bf2626f · bf2626f
1 parent 10c96e5
commit bf2626f
Show file tree

Hide file tree

Showing 2 changed files with 53 additions and 2 deletions.
diff --git a/x-pack/plugins/fleet/common/services/validate_package_policy.test.ts b/x-pack/plugins/fleet/common/services/validate_package_policy.test.ts
@@ -253,7 +253,7 @@ describe('Fleet - validatePackagePolicy()', () => {
           enabled: true,
           vars: {
             'foo-input-var-name': { value: undefined, type: 'text' },
-            'foo-input2-var-name': { value: '', type: 'text' },
+            'foo-input2-var-name': { value: undefined, type: 'text' },
             'foo-input3-var-name': { value: [], type: 'text' },
           },
           streams: [
@@ -555,6 +555,57 @@ describe('Fleet - validatePackagePolicy()', () => {
         inputs: null,
       });
     });
+
+    it('returns no errors when required field is present but empty', () => {
+      expect(
+        validatePackagePolicy(
+          {
+            ...validPackagePolicy,
+            inputs: [
+              {
+                type: 'foo',
+                policy_template: 'pkgPolicy1',
+                enabled: true,
+                vars: {
+                  'foo-input-var-name': { value: '', type: 'text' },
+                  'foo-input2-var-name': { value: '', type: 'text' },
+                  'foo-input3-var-name': { value: ['test'], type: 'text' },
+                },
+                streams: [
+                  {
+                    data_stream: { dataset: 'foo', type: 'logs' },
+                    enabled: true,
+                    vars: { 'var-name': { value: 'test_yaml: value', type: 'yaml' } },
+                  },
+                ],
+              },
+            ],
+          },
+          mockPackage,
+          safeLoad
+        )
+      ).toEqual({
+        name: null,
+        description: null,
+        namespace: null,
+        inputs: {
+          foo: {
+            streams: {
+              foo: {
+                vars: {
+                  'var-name': null,
+                },
+              },
+            },
+            vars: {
+              'foo-input-var-name': null,
+              'foo-input2-var-name': null,
+              'foo-input3-var-name': null,
+            },
+          },
+        },
+      });
+    });
   });
 
   describe('works for packages with multiple policy templates (aka integrations)', () => {

diff --git a/x-pack/plugins/fleet/common/services/validate_package_policy.ts b/x-pack/plugins/fleet/common/services/validate_package_policy.ts
@@ -217,7 +217,7 @@ export const validatePackagePolicyConfig = (
   }
 
   if (varDef.required) {
-    if (parsedValue === undefined || (typeof parsedValue === 'string' && !parsedValue)) {
+    if (parsedValue === undefined || (varDef.type === 'yaml' && parsedValue === '')) {
       errors.push(
         i18n.translate('xpack.fleet.packagePolicyValidation.requiredErrorMessage', {
           defaultMessage: '{fieldName} is required',