cartodb4/log4js-node
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Clusterred appender should consider the categories.

Browse Source 
It turns out that whenever the clusterred appender is used the log event is passed to all actual appenders.
The actual appender's category is ignored.

Signed-off-by: Vladimir Mitev <idalv@users.noreply.github.com>
This commit is contained in:
Vladimir Mitev 2014-05-20 13:45:46 +03:00
parent ca5272aacc
commit 2daf29b400
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/appenders/clustered.js
View File

@ -71,10 +71,13 @@ function createAppender(config) {
if (config.actualAppenders) { if (config.actualAppenders) {
var size = config.actualAppenders.length; var size = config.actualAppenders.length;
for(var i = 0; i < size; i++) { for(var i = 0; i < size; i++) {
if (!config.appenders[i].category || config.appenders[i].category === loggingEvent.categoryName) {
// Relying on the index is not a good practice but otherwise the change would have been bigger.
config.actualAppenders[i](loggingEvent); config.actualAppenders[i](loggingEvent);
} }
} }
} }
}
// Listen on new workers // Listen on new workers
cluster.on('fork', function(worker) { cluster.on('fork', function(worker) {