Rotate client log file daily

Signed-off-by: Rome Li <rome.li@microsoft.com>

Author: akaroml

package-lock.json

@@ -537,6 +537,7 @@
     "path-exists": "^3.0.0",
     "expand-home-dir": "^0.0.3",
     "glob": "^7.1.3",
-    "winston": "^3.2.1"
+    "winston": "^3.2.1",
+    "winston-daily-rotate-file": "^3.10.0"
   }
 }

package.json

@@ -21,6 +21,7 @@ import * as net from 'net';
 import { getJavaConfiguration } from './utils';
 import { onConfigurationChange, excludeProjectSettingsFiles } from './settings';
 import { logger, initializeLogFile } from './log';
+import glob = require('glob');
 
 let lastStatus;
 let languageClient: LanguageClient;
@@ -497,8 +498,21 @@ function openServerLogFile(workspacePath): Thenable<boolean> {
 	return openLogFile(serverLogFile, 'Could not open Java Language Server log file');
 }
 
-function openClientLogFile(logFile): Thenable<boolean> {
-	return openLogFile(logFile, 'Could not open Java extension log file');
+function openClientLogFile(logFile: string): Thenable<boolean> {
+	return new Promise((resolve) => {
+		const filename = path.basename(logFile);
+		const dirname = path.dirname(logFile);
+
+		// find out the newest one
+		glob(filename + '.*', { cwd: dirname }, (err, files) => {
+			if (!err && files.length > 0) {
+				files.sort();
+				logFile = path.join(dirname, files[files.length - 1]);
+			}
+
+			openLogFile(logFile, 'Could not open Java extension log file').then((result) => resolve(result));
+		});
+	});
 }
 
 

src/extension.ts

@@ -1,10 +1,12 @@
 import { createLogger, format, transports } from 'winston';
+import * as DailyRotateFile from 'winston-daily-rotate-file';
 
 export function initializeLogFile(filename: string) {
-	logger.add(new transports.File({
+	logger.add(new DailyRotateFile({
 		filename: filename,
-		maxsize: 100 * 1024, // 100k max size per file
-		maxFiles: 2 // the last 100k of logs are always available
+		datePattern: 'YYYY-MM-DD',
+		maxSize: '100k', // 100k max size per file
+		maxFiles: '2d' // retain logs of the last two days
 	}));
 }