mirror/appium
1
0
Fork 0
mirror of https://github.com/appium/appium.git synced 2026-02-20 18:30:11 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix(support): Print an empty string if no arguments are provided to a logging function (#20424)

Browse Source
This commit is contained in:
Mykola Mokhnach
2024-07-31 08:23:33 +02:00
committed by GitHub
parent 8413ccb4fa
commit 885570e7ca
2 changed files with 7 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
packages/logger/lib/log.ts
View File

@@ -245,9 +245,7 @@ export class Log extends EventEmitter implements Logger {
if (stack) {
messageArguments.unshift(`${stack}\n`);
}
const formattedMessage: string = messageArguments.length
? util.format(...messageArguments)
: '';
const formattedMessage = util.format(...messageArguments);
const m: MessageObject = {
id: this._id++,

8
packages/support/lib/logging.js
View File

@@ -67,8 +67,12 @@ export function getLogger(prefix = null) {
for (const level of LEVELS) {
wrappedLogger[level] = /** @param {any[]} args */ function (...args) {
const finalPrefix = getFinalPrefix(this.prefix, isDebugTimestampLoggingEnabled);
// @ts-ignore This is OK
logger[level](finalPrefix, ...args);
if (args.length) {
// @ts-ignore This is OK
logger[level](finalPrefix, ...args);
} else {
logger[level](finalPrefix, '');
}
};
}
if (!usingGlobalLog) {