Merge pull request #184 from thomasf/issue158

Kind of a hack but fixes issue #158
pull/203/merge
Peter 'Pita' Martischka 2011-11-14 21:35:37 -08:00
commit 6d190f517e
1 changed files with 4 additions and 1 deletions

View File

@ -91,7 +91,10 @@ async.waterfall([
var httpLogger = log4js.getLogger("http");
app.configure(function()
{
app.use(log4js.connectLogger(httpLogger, { level: log4js.levels.INFO, format: ':status, :method :url'}));
// If the log level specified in the config file is WARN or ERROR the application server never starts listening to requests as reported in issue #158.
// Not installing the log4js connect logger when the log level has a higher severity than INFO since it would not log at that level anyway.
if (!(settings.loglevel === "WARN" || settings.loglevel == "ERROR"))
app.use(log4js.connectLogger(httpLogger, { level: log4js.levels.INFO, format: ':status, :method :url'}));
app.use(express.cookieParser());
});