Don't set a min-length on encryption key for reportin (#69827) (#69859)

elastic · Jun 25, 2020 · 0138448 · 0138448
1 parent 8bc04a9
commit 0138448
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/x-pack/plugins/reporting/server/config/schema.test.ts b/x-pack/plugins/reporting/server/config/schema.test.ts
@@ -112,6 +112,8 @@ describe('Reporting Config Schema', () => {
         .encryptionKey
     ).toBe('qqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqq');
 
+    expect(ConfigSchema.validate({ encryptionKey: 'weaksauce' }).encryptionKey).toBe('weaksauce');
+
     // disableSandbox
     expect(
       ConfigSchema.validate({ capture: { browser: { chromium: { disableSandbox: true } } } })

diff --git a/x-pack/plugins/reporting/server/config/schema.ts b/x-pack/plugins/reporting/server/config/schema.ts
@@ -136,8 +136,8 @@ const CsvSchema = schema.object({
 const EncryptionKeySchema = schema.conditional(
   schema.contextRef('dist'),
   true,
-  schema.maybe(schema.string({ minLength: 32 })), // default value is dynamic in createConfig$
-  schema.string({ minLength: 32, defaultValue: 'a'.repeat(32) })
+  schema.maybe(schema.string()), // default value is dynamic in createConfig$
+  schema.string({ defaultValue: 'a'.repeat(32) })
 );
 
 const RolesSchema = schema.object({