Remove string as a valid registry var type value (elastic#94174) (e…

…lastic#94751) Co-authored-by: Kibana Machine <[email protected]> Co-authored-by: Jen Huang <[email protected]>
rudolf · Mar 16, 2021 · 9b1a579 · 9b1a579
1 parent d52faa3
commit 9b1a579
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 19 deletions.
diff --git a/x-pack/plugins/fleet/common/types/models/epm.ts b/x-pack/plugins/fleet/common/types/models/epm.ts
@@ -272,6 +272,7 @@ export interface RegistryElasticsearch {
   'index_template.mappings'?: object;
 }
 
+export type RegistryVarType = 'integer' | 'bool' | 'password' | 'text' | 'yaml' | 'string';
 export enum RegistryVarsEntryKeys {
   name = 'name',
   title = 'title',
@@ -284,7 +285,6 @@ export enum RegistryVarsEntryKeys {
   os = 'os',
 }
 
-export type RegistryVarType = 'integer' | 'bool' | 'password' | 'text' | 'yaml' | 'string';
 // EPR types this as `[]map[string]interface{}`
 // which means the official/possible type is Record<string, any>
 // but we effectively only see this shape

diff --git a/...leet/sections/agent_policy/create_package_policy_page/services/validate_package_policy.ts b/...leet/sections/agent_policy/create_package_policy_page/services/validate_package_policy.ts
@@ -228,11 +228,7 @@ export const validatePackagePolicyConfig = (
         })
       );
     }
-    if (
-      (varDef.type === 'text' || varDef.type === 'string') &&
-      parsedValue &&
-      Array.isArray(parsedValue)
-    ) {
+    if (varDef.type === 'text' && parsedValue && Array.isArray(parsedValue)) {
       const invalidStrings = parsedValue.filter((cand) => /^[*&]/.test(cand));
       // only show one error if multiple strings in array are invalid
       if (invalidStrings.length > 0) {
@@ -246,11 +242,7 @@ export const validatePackagePolicyConfig = (
     }
   }
 
-  if (
-    (varDef.type === 'text' || varDef.type === 'string') &&
-    parsedValue &&
-    !Array.isArray(parsedValue)
-  ) {
+  if (varDef.type === 'text' && parsedValue && !Array.isArray(parsedValue)) {
     if (/^[*&]/.test(parsedValue)) {
       errors.push(
         i18n.translate('xpack.fleet.packagePolicyValidation.quoteStringErrorMessage', {

diff --git a/x-pack/plugins/fleet/server/services/epm/agent/agent.test.ts b/x-pack/plugins/fleet/server/services/epm/agent/agent.test.ts
@@ -193,14 +193,14 @@ my-package:
         {{{ search }}} | streamstats`;
 
     const vars = {
-      asteriskOnly: { value: '"*"', type: 'string' },
-      startsWithAsterisk: { value: '"*lala"', type: 'string' },
-      numeric: { value: '100', type: 'string' },
-      mixed: { value: '1s', type: 'string' },
-      a: { value: '/opt/package/*', type: 'string' },
-      b: { value: '/logs/my.log*', type: 'string' },
-      c: { value: '/opt/*/package/', type: 'string' },
-      d: { value: 'logs/*my.log', type: 'string' },
+      asteriskOnly: { value: '"*"', type: 'text' },
+      startsWithAsterisk: { value: '"*lala"', type: 'text' },
+      numeric: { value: '100', type: 'text' },
+      mixed: { value: '1s', type: 'text' },
+      a: { value: '/opt/package/*', type: 'text' },
+      b: { value: '/logs/my.log*', type: 'text' },
+      c: { value: '/opt/*/package/', type: 'text' },
+      d: { value: 'logs/*my.log', type: 'text' },
       search: { value: 'search sourcetype="access*"', type: 'text' },
     };