[Bug]: forceConsole may cause circular reference if console.log is redirected to Winston

[neilenns]

🔎 Search Terms

forceConsole

The problem

As mentioned in #2276 (comment), sometimes people will redirect the console.log method to a Winston logger to ensure all logged messages get captured:

console.log = logger.log.bind(logger)

Apparently, this can cause a circular reference issue, since the Winston logger will attempt to log using console.log, but that's been redirected to be the logger itself.

What version of Winston presents the issue?

v3.14.1

What version of Node are you using?

v20.11.0

If this worked in a previous version of Winston, which was it?

v3.14.0

Minimum Working Example

const mainLogger = winston.createLogger({
  level: "debug",
  transports: [
    new winston.transports.Console({ forceConsole: true }),
  ],
});

console.log = mainLogger.log.bind(mainLogger);

mainLogger.log("info", "Oops");

Additional information

PR to fix this will be opened shortly.