docs: warn when the log level is TRACE or ALL as the CLI might print sensitive data

DimitarTachev · rosen-vladimirov · commit 50dfa5c55f15 · 2020-02-20T13:41:22.000+02:00
diff --git a/lib/common/logger/logger.ts b/lib/common/logger/logger.ts
@@ -48,6 +48,9 @@ export class Logger implements ILogger {
 		log4js.configure({ appenders, categories });
 
 		this.log4jsLogger = log4js.getLogger();
+		if (level === LoggerLevel.TRACE || level === LoggerLevel.ALL) {
+			this.warn(`The "${level}" log level might print some sensitive data like secrets or access tokens in request URLs. Be careful when you share this output.`, { wrapMessageWithBorders: true });
+		}
 	}
 
 	public initializeCliLogger(opts?: ILoggerOptions): void {

Original file line number	Diff line number	Diff line change
`@@ -48,6 +48,9 @@ export class Logger implements ILogger {`
`48`	`48`	`log4js.configure({ appenders, categories });`
`49`	`49`
`50`	`50`	`this.log4jsLogger = log4js.getLogger();`
	`51`	`+ if (level === LoggerLevel.TRACE \|\| level === LoggerLevel.ALL) {`
	`52`	+ this.warn(`The "${level}" log level might print some sensitive data like secrets or access tokens in request URLs. Be careful when you share this output.`, { wrapMessageWithBorders: true });
	`53`	`+ }`
`51`	`54`	`}`
`52`	`55`
`53`	`56`	`public initializeCliLogger(opts?: ILoggerOptions): void {`