1.0.8 / 2021-07-14

==================

* Doh, config must be injected (figures)  - @craigparra

Author: craigparra

ConfigurableLogger.js

@@ -2,9 +2,9 @@ const CONFIG = require('config');
 const DelegatingLogger = require('./DelegatingLogger');
 
 module.exports = class ConfigurableLogger extends DelegatingLogger {
-  constructor(provider, category, configPath, registry, config) {
+  constructor(config, provider, category, configPath, registry) {
     super(provider);
-    this.config = config || CONFIG;
+    this.config = config;
     this.category = category;
     this.configPath = configPath;
     this.registry = registry;

History.md

@@ -1,3 +1,8 @@
+1.0.8 / 2021-07-14
+==================
+
+* Doh, config must be injected (figures)  - @craigparra
+  
 1.0.7 / 2021-07-14
 ==================
 
@@ -14,7 +19,7 @@
 * Fixed usage example to use ConsoleLogger - @craigparra
 * Added slf4j keyword - @craigparra
 
-``1.0.4 / 2021-07-12
+1.0.4 / 2021-07-12
 ==================
 
 * Fixed JSON outputs to proper keyval pairs, plus category output - @craigparra

LoggerFactory.js

@@ -5,8 +5,8 @@ const LoggerRegistry = require('./LoggerRegistry');
 module.exports = class LoggerFactory {
     static loggerRegistry = new LoggerRegistry();
 
-    static getLogger(category, provider, configPath, registry) {
-      return new ConfigurableLogger(
+    static getLogger(config, category, provider, configPath, registry) {
+      return new ConfigurableLogger(config,
         provider || new ConsoleLogger(category),
         category,
         configPath,

README.md

@@ -19,8 +19,9 @@ To use the module, import the LoggerFactory and call the `getLogger` function wi
 requires path is a sensible choice).
 
 ```javascript
+const {config} = require('config');
 const {LoggerFactory} = require('@alt-javascript/logger');
-const logger = LoggerFactory.getLogger('@myorg/mypackage/MyModule');
+const logger = LoggerFactory.getLogger(config,'@myorg/mypackage/MyModule');
 
 logger.info('Hello world!');
 ```
@@ -76,8 +77,9 @@ function, passing it your winston options (nullish options will fall back to def
     
 
 ```javascript
+const {config} = require('config');
 const {LoggerFactory,WinstonLogger} = require('@alt-javascript/logger');
-const logger = LoggerFactory.getLogger('@myorg/mypackage/MyModule', new WinstonLogger({/*mywinstonoptions*/}));
+const logger = LoggerFactory.getLogger(config,'@myorg/mypackage/MyModule', new WinstonLogger({/*mywinstonoptions*/}));
 
 logger.info('Hello world!');
 ```
@@ -98,9 +100,10 @@ logger.info('Hello world!');
 Testing loggers is hard, and testability is a first class concern at @alt-javascript so the module exports an EphemeralLogger and EphemeralLogSink that will capture log lines that can be asserted.
 
 ```javascript
+const {config} = require('config');
 const {LoggerFactory,EphemeralLogger} = require('@alt-javascript/logger');
 const ephemeralLogger = new EphemeralLogger('@myorg/mypackage/MyModule');
-const logger = LoggerFactory.getLogger('@myorg/mypackage/MyModule', ephemeralLogger);
+const logger = LoggerFactory.getLogger(config,'@myorg/mypackage/MyModule', ephemeralLogger);
 
 logger.info('Hello world!');
 

package.json

@@ -2,8 +2,18 @@
   "name": "@alt-javascript/logger",
   "version": "1.0.6",
   "description": "A simple configurable logging facade for javascript.",
-  "author" : "Craig Parravicini",
-  "keywords": ["log","logger","loglevel", "loggerlevel","LogFactory","facade","config","configurable","slf4j"],
+  "author": "",
+  "keywords": [
+    "log",
+    "logger",
+    "loglevel",
+    "loggerlevel",
+    "LogFactory",
+    "facade",
+    "config",
+    "configurable",
+    "slf4j"
+  ],
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
   },
@@ -22,7 +32,6 @@
     "mocha": "mocha --require test/fixtures",
     "nyc": "nyc --reporter=json-summary --reporter=text-summary --reporter=lcov npm run test"
   },
-  "author": "",
   "license": "MIT",
   "dependencies": {
     "config": "^3.3.6",

test/console_spec.js

@@ -1,13 +1,13 @@
 const { assert } = require('chai');
 const LoggerFactory = require('../LoggerFactory');
 const LoggerLevel = require('../LoggerLevel');
-
-const logger = LoggerFactory.getLogger('@alt-javascript/logger/test/console_spec');
-const defaultLogger = LoggerFactory.getLogger();
+const config = require('config');
+const logger = LoggerFactory.getLogger(config,'@alt-javascript/logger/test/console_spec');
+const defaultLogger = LoggerFactory.getLogger(config);
 defaultLogger.registry.add('logging.level./', LoggerLevel.DEBUG);
-const altDefaultLogger = LoggerFactory.getLogger();
+const altDefaultLogger = LoggerFactory.getLogger(config);
 
-const verbose = LoggerFactory.getLogger('@alt-javascript/logger/test/console_spec/verbose');
+const verbose = LoggerFactory.getLogger(config,'@alt-javascript/logger/test/console_spec/verbose');
 
 assert.isFalse(logger.isVerboseEnabled(), 'Verbose is false');
 logger.setLevel(null);

test/winston_spec.js

@@ -1,10 +1,11 @@
+const config = require('config');
 const { assert } = require('chai');
 const LoggerFactory = require('../LoggerFactory');
 const LoggerLevel = require('../LoggerLevel');
 const WinstonLogger = require('../WinstonLogger');
 
-const logger = LoggerFactory.getLogger('@alt-javascript/logger/test/winston_spec', new WinstonLogger('@alt-javascript/logger/test/winston_spec'));
-const verbose = LoggerFactory.getLogger('@alt-javascript/logger/test/winston_spec/verbose', new WinstonLogger('@alt-javascript/logger/test/winston_spec/verbose'));
+const logger = LoggerFactory.getLogger(config,'@alt-javascript/logger/test/winston_spec', new WinstonLogger('@alt-javascript/logger/test/winston_spec'));
+const verbose = LoggerFactory.getLogger(config,'@alt-javascript/logger/test/winston_spec/verbose', new WinstonLogger('@alt-javascript/logger/test/winston_spec/verbose'));
 
 assert.isTrue(logger.isVerboseEnabled(), 'Verbose is false');
 logger.setLevel(null);