archive-gh-me/log4js-node
1
0
Fork 0
mirror of https://github.com/log4js-node/log4js-node.git synced 2025-12-08 19:26:01 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #312 from ezbin/feature-smtp-transport-plugins

Browse Source 
Feature - Add support for nodemailer transport plugins
This commit is contained in:
Gareth Jones 2015-09-22 08:26:08 +10:00
commit 1ea98b3836
2 changed files with 77 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
lib/appenders/smtp.js
View File

@ -30,7 +30,8 @@ function smtpAppender(config, layout) {
function sendBuffer() {
if (logEventBuffer.length > 0) {
var transport = mailer.createTransport(config.SMTP);
var transportOpts = getTransportOptions(config);
var transport = mailer.createTransport(transportOpts);
var firstEvent = logEventBuffer[0];
var body = "";
var count = logEventBuffer.length;
@ -72,6 +73,20 @@ function smtpAppender(config, layout) {
}
}
function getTransportOptions(config) {
var transportOpts = null;
if( config.SMTP ) {
transportOpts = config.SMTP;
} else if( config.transport ) {
var plugin = config.transport.plugin || 'smtp';
var transportModule = 'nodemailer-' + plugin + '-transport';
var transporter = require( transportModule );
transportOpts = transporter( config.transport.options );
}
return transportOpts;
}
return function(loggingEvent) {
unsentCount++;
logEventBuffer.push(loggingEvent);

61
test/smtpAppender-test.js
View File

@ -38,9 +38,14 @@ function setupLogging(category, options) {
}
};
var fakeTransportPlugin = function () {
};
var smtpModule = sandbox.require('../lib/appenders/smtp', {
requires: {
'nodemailer': fakeMailer,
'nodemailer-sendmail-transport': fakeTransportPlugin,
'nodemailer-smtp-transport': fakeTransportPlugin,
'../layouts': fakeLayouts
},
globals: {
@ -222,5 +227,61 @@ vows.describe('log4js smtpAppender').addBatch({
assert.equal(cons.errors[0].msg, "log4js.smtpAppender - Error happened");
assert.equal(cons.errors[0].value.message, 'oh noes');
}
},
'transport full config': {
topic: function() {
var setup = setupLogging('transport full config', {
recipients: 'recipient@domain.com',
transport: {
plugin: 'sendmail',
options: {
path: '/usr/sbin/sendmail'
}
}
});
setup.logger.info('Log event #1');
return setup;
},
'there should be one message only': function (result) {
assert.equal(result.results.length, 1);
},
'message should contain proper data': function (result) {
checkMessages(result);
}
},
'transport no-options config': {
topic: function() {
var setup = setupLogging('transport no-options config', {
recipients: 'recipient@domain.com',
transport: {
plugin: 'sendmail'
}
});
setup.logger.info('Log event #1');
return setup;
},
'there should be one message only': function (result) {
assert.equal(result.results.length, 1);
},
'message should contain proper data': function (result) {
checkMessages(result);
}
},
'transport no-plugin config': {
topic: function() {
var setup = setupLogging('transport no-plugin config', {
recipients: 'recipient@domain.com',
transport: {
}
});
setup.logger.info('Log event #1');
return setup;
},
'there should be one message only': function (result) {
assert.equal(result.results.length, 1);
},
'message should contain proper data': function (result) {
checkMessages(result);
}
}
}).export(module);