diff --git a/lib/appenders/dateFile.js b/lib/appenders/dateFile.js index 8999c74..76de634 100644 --- a/lib/appenders/dateFile.js +++ b/lib/appenders/dateFile.js @@ -10,7 +10,8 @@ function openTheStream(filename, pattern, options) { options ); stream.on('error', (err) => { - console.error('log4js.dateFileAppender - Writing to file %s, error happened ', filename, err); // eslint-disable-line + // eslint-disable-next-line no-console + console.error('log4js.dateFileAppender - Writing to file %s, error happened ', filename, err); }); stream.on("drain", () => { process.emit("log4js:pause", false); diff --git a/lib/appenders/file.js b/lib/appenders/file.js index 2680fc1..73dad71 100644 --- a/lib/appenders/file.js +++ b/lib/appenders/file.js @@ -50,7 +50,8 @@ function fileAppender(file, layout, logSize, numBackups, options, timezoneOffset opt ); stream.on('error', (err) => { - console.error('log4js.fileAppender - Writing to file %s, error happened ', filePath, err); // eslint-disable-line + // eslint-disable-next-line no-console + console.error('log4js.fileAppender - Writing to file %s, error happened ', filePath, err); }); stream.on('drain', () => { process.emit("log4js:pause", false); diff --git a/lib/appenders/index.js b/lib/appenders/index.js index 2039dab..b9100b1 100644 --- a/lib/appenders/index.js +++ b/lib/appenders/index.js @@ -24,7 +24,8 @@ const appenders = new Map(); const tryLoading = (modulePath, config) => { debug('Loading module from ', modulePath); try { - return require(modulePath); // eslint-disable-line + // eslint-disable-next-line global-require, import/no-dynamic-require + return require(modulePath); } catch (e) { // if the module was found, and we still got an error, then raise it configuration.throwExceptionIf( @@ -83,7 +84,7 @@ const createAppender = (name, config) => { } debug(`${name}: clustering.isMaster ? ${clustering.isMaster()}`); - debug(`${name}: appenderModule is ${require('util').inspect(appenderModule)}`); // eslint-disable-line + debug(`${name}: appenderModule is ${require('util').inspect(appenderModule)}`); // eslint-disable-line global-require return clustering.onlyOnMaster(() => { debug(`calling appenderModule.configure for ${name} / ${appenderConfig.type}`); return appenderModule.configure( diff --git a/lib/clustering.js b/lib/clustering.js index 61c6ff1..fefe976 100644 --- a/lib/clustering.js +++ b/lib/clustering.js @@ -5,7 +5,8 @@ const configuration = require("./configuration"); let disabled = false; let cluster = null; try { - cluster = require("cluster"); // eslint-disable-line + // eslint-disable-next-line global-require + cluster = require("cluster"); } catch (e) { debug("cluster module not present"); disabled = true; diff --git a/test/tap/LoggingEvent-test.js b/test/tap/LoggingEvent-test.js index c6af015..a9f6487 100644 --- a/test/tap/LoggingEvent-test.js +++ b/test/tap/LoggingEvent-test.js @@ -48,7 +48,7 @@ test("LoggingEvent", batch => { batch.test("Should correct construct with/without location info", t => { // console.log([Error('123').stack.split('\n').slice(1).join('\n')]) const callStack = - " at repl:1:14\n at ContextifyScript.Script.runInThisContext (vm.js:50:33)\n at REPLServer.defaultEval (repl.js:240:29)\n at bound (domain.js:301:14)\n at REPLServer.runBound [as eval] (domain.js:314:12)\n at REPLServer.onLine (repl.js:468:10)\n at emitOne (events.js:121:20)\n at REPLServer.emit (events.js:211:7)\n at REPLServer.Interface._onLine (readline.js:280:10)\n at REPLServer.Interface._line (readline.js:629:8)"; // eslint-disable-line + " at repl:1:14\n at ContextifyScript.Script.runInThisContext (vm.js:50:33)\n at REPLServer.defaultEval (repl.js:240:29)\n at bound (domain.js:301:14)\n at REPLServer.runBound [as eval] (domain.js:314:12)\n at REPLServer.onLine (repl.js:468:10)\n at emitOne (events.js:121:20)\n at REPLServer.emit (events.js:211:7)\n at REPLServer.Interface._onLine (readline.js:280:10)\n at REPLServer.Interface._line (readline.js:629:8)"; // eslint-disable-line max-len const fileName = "/log4js-node/test/tap/layouts-test.js"; const lineNumber = 1; const columnNumber = 14; diff --git a/test/tap/layouts-test.js b/test/tap/layouts-test.js index f2f1843..855b6c9 100644 --- a/test/tap/layouts-test.js +++ b/test/tap/layouts-test.js @@ -257,7 +257,7 @@ test("log4js layouts", batch => { // console.log([Error('123').stack.split('\n').slice(1).join('\n')]) const callStack = - " at repl:1:14\n at ContextifyScript.Script.runInThisContext (vm.js:50:33)\n at REPLServer.defaultEval (repl.js:240:29)\n at bound (domain.js:301:14)\n at REPLServer.runBound [as eval] (domain.js:314:12)\n at REPLServer.onLine (repl.js:468:10)\n at emitOne (events.js:121:20)\n at REPLServer.emit (events.js:211:7)\n at REPLServer.Interface._onLine (readline.js:280:10)\n at REPLServer.Interface._line (readline.js:629:8)"; // eslint-disable-line + " at repl:1:14\n at ContextifyScript.Script.runInThisContext (vm.js:50:33)\n at REPLServer.defaultEval (repl.js:240:29)\n at bound (domain.js:301:14)\n at REPLServer.runBound [as eval] (domain.js:314:12)\n at REPLServer.onLine (repl.js:468:10)\n at emitOne (events.js:121:20)\n at REPLServer.emit (events.js:211:7)\n at REPLServer.Interface._onLine (readline.js:280:10)\n at REPLServer.Interface._line (readline.js:629:8)"; // eslint-disable-line max-len const fileName = path.normalize("/log4js-node/test/tap/layouts-test.js"); const lineNumber = 1; const columnNumber = 14;