From 00c62c7fa6a4a79dd3e61be5be578bc9d408ac1f Mon Sep 17 00:00:00 2001
From: Jakub Chmura <jakub@jakubchmura.pl>
Date: Mon, 25 Aug 2014 19:48:15 +0200
Subject: [PATCH 1/2] Adding maxLevel to levelFilter.

This optional parameter specifies what maximum level of event is accepted by the filter.
---
 lib/appenders/logLevelFilter.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/lib/appenders/logLevelFilter.js b/lib/appenders/logLevelFilter.js
index ddbb61c..b771d7c 100644
--- a/lib/appenders/logLevelFilter.js
+++ b/lib/appenders/logLevelFilter.js
@@ -2,10 +2,12 @@
 var levels = require('../levels')
 , log4js = require('../log4js');
 
-function logLevelFilter (levelString, appender) {
-  var level = levels.toLevel(levelString);
+function logLevelFilter (minLevelString, maxLevelString, appender) {
+  var minLevel = levels.toLevel(minLevelString);
+  var maxLevel = levels.toLevel(maxLevelString, levels.FATAL);
   return function(logEvent) {
-    if (logEvent.level.isGreaterThanOrEqualTo(level)) {
+      var eventLevel = logEvent.level;
+      if (eventLevel.isGreaterThanOrEqualTo(minLevel) && eventLevel.isLessThanOrEqualTo(maxLevel)) {
       appender(logEvent);
     }
   };
@@ -14,7 +16,7 @@ function logLevelFilter (levelString, appender) {
 function configure(config) {
   log4js.loadAppender(config.appender.type);
   var appender = log4js.appenderMakers[config.appender.type](config.appender);
-  return logLevelFilter(config.level, appender);
+  return logLevelFilter(config.level, config.maxLevel, appender);
 }
 
 exports.appender = logLevelFilter;

From ade6dd8ea03a5948ebd23d0362dc21ea5e156099 Mon Sep 17 00:00:00 2001
From: Jakub Chmura <jakub@jakubchmura.pl>
Date: Mon, 25 Aug 2014 20:56:37 +0200
Subject: [PATCH 2/2] Adjusting and adding new tests for logLevelFilter to
 handle the maxLevel option.

---
 test/logLevelFilter-test.js   | 25 +++++++++++++++++++------
 test/with-logLevelFilter.json | 15 ++++++++++++++-
 2 files changed, 33 insertions(+), 7 deletions(-)

diff --git a/test/logLevelFilter-test.js b/test/logLevelFilter-test.js
index 2280a96..b3deb05 100644
--- a/test/logLevelFilter-test.js
+++ b/test/logLevelFilter-test.js
@@ -22,6 +22,7 @@ vows.describe('log4js logLevelFilter').addBatch({
         require('../lib/appenders/logLevelFilter')
           .appender(
             'ERROR',
+            undefined,
             function(evt) { logEvents.push(evt); }
           ),
         "logLevelTest"
@@ -48,13 +49,16 @@ vows.describe('log4js logLevelFilter').addBatch({
 
       remove(__dirname + '/logLevelFilter.log');
       remove(__dirname + '/logLevelFilter-warnings.log');
+      remove(__dirname + '/logLevelFilter-debugs.log');
 
       log4js.configure('test/with-logLevelFilter.json');
       logger = log4js.getLogger("tests");
-      logger.info('main');
-      logger.error('both');
-      logger.warn('both');
-      logger.debug('main');
+      logger.debug('debug');
+      logger.info('info');
+      logger.error('error');
+      logger.warn('warn');
+      logger.debug('debug');
+      logger.trace('trace');
       //wait for the file system to catch up
       setTimeout(this.callback, 500);
     },
@@ -64,7 +68,7 @@ vows.describe('log4js logLevelFilter').addBatch({
       },
       'should contain all log messages': function (contents) {
         var messages = contents.trim().split(EOL);
-        assert.deepEqual(messages, ['main','both','both','main']);
+        assert.deepEqual(messages, ['debug','info','error','warn','debug','trace']);
       }
     },
     'tmp-tests-warnings.log': {
@@ -73,7 +77,16 @@ vows.describe('log4js logLevelFilter').addBatch({
       },
       'should contain only error and warning log messages': function(contents) {
         var messages = contents.trim().split(EOL);
-        assert.deepEqual(messages, ['both','both']);
+        assert.deepEqual(messages, ['error','warn']);
+      }
+    },
+    'tmp-tests-debugs.log': {
+      topic: function() {
+        fs.readFile(__dirname + '/logLevelFilter-debugs.log','utf8',this.callback);
+      },
+      'should contain only trace and debug log messages': function(contents) {
+        var messages = contents.trim().split(EOL);
+        assert.deepEqual(messages, ['debug','debug','trace']);
       }
     }
   }
diff --git a/test/with-logLevelFilter.json b/test/with-logLevelFilter.json
index d564ace..7bcd8ad 100644
--- a/test/with-logLevelFilter.json
+++ b/test/with-logLevelFilter.json
@@ -12,6 +12,19 @@
         } 
       }
     },
+    {
+      "category": "tests",
+      "type": "logLevelFilter",
+      "level": "TRACE",
+      "maxLevel": "DEBUG",
+      "appender": {
+        "type": "file",
+        "filename": "test/logLevelFilter-debugs.log",
+        "layout": {
+          "type": "messagePassThrough"
+          }
+        }
+    },
     { 
       "category": "tests", 
       "type": "file",
@@ -23,6 +36,6 @@
   ],
   
   "levels": {
-    "tests":  "DEBUG"
+    "tests":  "TRACE"
   }
 }