github-mirrors/kibana
1
0
Fork 0
mirror of https://github.com/elastic/kibana.git synced 2025-04-23 09:19:04 -04:00
Code Issues Projects Releases Packages Wiki Activity

Remove string as a valid registry var type value (#94174) (#94751)

Browse source 
Co-authored-by: Kibana Machine <42973632+kibanamachine@users.noreply.github.com>

Co-authored-by: Jen Huang <its.jenetic@gmail.com>
This commit is contained in:
Kibana Machine 2021-03-16 17:42:31 -04:00 • committed by GitHub
parent d52faa3509
commit 9b1a579afd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

2
x-pack/plugins/fleet/common/types/models/epm.ts
View file

@ -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

12
x-pack/plugins/fleet/public/applications/fleet/sections/agent_policy/create_package_policy_page/services/validate_package_policy.ts
View file

@ -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', {

16
x-pack/plugins/fleet/server/services/epm/agent/agent.test.ts
View file

@ -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' },
};