Skip to content

Commit ba9aed4

Browse files
author
Joel Griffith
authored
Don't set a min-length on encryption key for reportin (#69827)
1 parent a89fa3c commit ba9aed4

2 files changed

Lines changed: 4 additions & 2 deletions

File tree

x-pack/plugins/reporting/server/config/schema.test.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,8 @@ describe('Reporting Config Schema', () => {
112112
.encryptionKey
113113
).toBe('qqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqq');
114114

115+
expect(ConfigSchema.validate({ encryptionKey: 'weaksauce' }).encryptionKey).toBe('weaksauce');
116+
115117
// disableSandbox
116118
expect(
117119
ConfigSchema.validate({ capture: { browser: { chromium: { disableSandbox: true } } } })

x-pack/plugins/reporting/server/config/schema.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,8 +136,8 @@ const CsvSchema = schema.object({
136136
const EncryptionKeySchema = schema.conditional(
137137
schema.contextRef('dist'),
138138
true,
139-
schema.maybe(schema.string({ minLength: 32 })), // default value is dynamic in createConfig$
140-
schema.string({ minLength: 32, defaultValue: 'a'.repeat(32) })
139+
schema.maybe(schema.string()), // default value is dynamic in createConfig$
140+
schema.string({ defaultValue: 'a'.repeat(32) })
141141
);
142142

143143
const RolesSchema = schema.object({

0 commit comments

Comments
 (0)