From 96f1edb9b0ce80dbe79e4cf8ad911e0eb9fae32e Mon Sep 17 00:00:00 2001
From: Mikhail Shustov <restrry@gmail.com>
Date: Wed, 12 Feb 2020 14:26:02 +0100
Subject: [PATCH] remove file from pre-defined appenders.

file name is required. let users to setup everything
---
 .../server/logging/logging_config.test.ts     | 19 ++-----------------
 src/core/server/logging/logging_config.ts     |  7 -------
 2 files changed, 2 insertions(+), 24 deletions(-)

diff --git a/src/core/server/logging/logging_config.test.ts b/src/core/server/logging/logging_config.test.ts
index b3631abb9ff00..75f571d34c25c 100644
--- a/src/core/server/logging/logging_config.test.ts
+++ b/src/core/server/logging/logging_config.test.ts
@@ -59,7 +59,7 @@ test('`getLoggerContext()` returns correct joined context name.', () => {
 test('correctly fills in default config.', () => {
   const configValue = new LoggingConfig(config.schema.validate({}));
 
-  expect(configValue.appenders.size).toBe(3);
+  expect(configValue.appenders.size).toBe(2);
 
   expect(configValue.appenders.get('default')).toEqual({
     kind: 'console',
@@ -69,10 +69,6 @@ test('correctly fills in default config.', () => {
     kind: 'console',
     layout: { kind: 'pattern', highlight: true },
   });
-  expect(configValue.appenders.get('file')).toEqual({
-    kind: 'file',
-    layout: { kind: 'pattern', highlight: false },
-  });
 });
 
 test('correctly fills in custom `appenders` config.', () => {
@@ -83,16 +79,11 @@ test('correctly fills in custom `appenders` config.', () => {
           kind: 'console',
           layout: { kind: 'pattern' },
         },
-        file: {
-          kind: 'file',
-          layout: { kind: 'pattern' },
-          path: 'path',
-        },
       },
     })
   );
 
-  expect(configValue.appenders.size).toBe(3);
+  expect(configValue.appenders.size).toBe(2);
 
   expect(configValue.appenders.get('default')).toEqual({
     kind: 'console',
@@ -103,12 +94,6 @@ test('correctly fills in custom `appenders` config.', () => {
     kind: 'console',
     layout: { kind: 'pattern' },
   });
-
-  expect(configValue.appenders.get('file')).toEqual({
-    kind: 'file',
-    layout: { kind: 'pattern' },
-    path: 'path',
-  });
 });
 
 test('correctly fills in default `loggers` config.', () => {
diff --git a/src/core/server/logging/logging_config.ts b/src/core/server/logging/logging_config.ts
index f1fbf787737b4..8f80be7d79cb1 100644
--- a/src/core/server/logging/logging_config.ts
+++ b/src/core/server/logging/logging_config.ts
@@ -140,13 +140,6 @@ export class LoggingConfig {
         layout: { kind: 'pattern', highlight: true },
       } as AppenderConfigType,
     ],
-    [
-      'file',
-      {
-        kind: 'file',
-        layout: { kind: 'pattern', highlight: false },
-      } as AppenderConfigType,
-    ],
   ]);
 
   /**