Fix: Winston Logger string interpolation (#5729)

spec/WinstonLoggerAdapter.spec.js

@@ -43,6 +43,22 @@ describe('info logs', () => {
       }
     );
   });
+
+  it('info logs should interpolate string', async () => {
+    const winstonLoggerAdapter = new WinstonLoggerAdapter();
+    winstonLoggerAdapter.log('info', 'testing info logs with %s', 'replace');
+    const results = await winstonLoggerAdapter.query({
+      from: new Date(Date.now() - 500),
+      size: 100,
+      level: 'info',
+      order: 'desc',
+    });
+    expect(results.length > 0).toBeTruthy();
+    const log = results.find(
+      x => x.message === 'testing info logs with replace'
+    );
+    expect(log);
+  });
 });
 
 describe('error logs', () => {
@@ -82,6 +98,21 @@ describe('error logs', () => {
       }
     );
   });
+
+  it('error logs should interpolate string', async () => {
+    const winstonLoggerAdapter = new WinstonLoggerAdapter();
+    winstonLoggerAdapter.log('error', 'testing error logs with %s', 'replace');
+    const results = await winstonLoggerAdapter.query({
+      from: new Date(Date.now() - 500),
+      size: 100,
+      level: 'error',
+    });
+    expect(results.length > 0).toBeTruthy();
+    const log = results.find(
+      x => x.message === 'testing error logs with replace'
+    );
+    expect(log);
+  });
 });
 
 describe('verbose logs', () => {
@@ -129,4 +160,24 @@ describe('verbose logs', () => {
         done();
       });
   });
+
+  it('verbose logs should interpolate string', async () => {
+    await reconfigureServer({ verbose: true });
+    const winstonLoggerAdapter = new WinstonLoggerAdapter();
+    winstonLoggerAdapter.log(
+      'verbose',
+      'testing verbose logs with %s',
+      'replace'
+    );
+    const results = await winstonLoggerAdapter.query({
+      from: new Date(Date.now() - 500),
+      size: 100,
+      level: 'verbose',
+    });
+    expect(results.length > 0).toBeTruthy();
+    const log = results.find(
+      x => x.message === 'testing verbose logs with replace'
+    );
+    expect(log);
+  });
 });

src/Adapters/Logger/WinstonLogger.js

@@ -32,7 +32,11 @@ function configureTransports(options) {
           {
             filename: 'parse-server.err',
             json: true,
-            format: format.combine(format.timestamp(), format.json()),
+            format: format.combine(
+              format.timestamp(),
+              format.splat(),
+              format.json()
+            ),
           },
           options,
           { level: 'error' }