wpstudio/hiring-test-one
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

fixes #18, '--log-format none' now works correctly

Browse Source
This commit is contained in:
Lloyd Brookes
2015-08-25 22:16:19 +01:00
parent 0f3b7a9e7f
commit ae592246b4
1 changed files with 11 additions and 15 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
bin/ws.js
View File

@ -110,7 +110,7 @@ function launchServer(){
next();
});
app.use(getLogger());
if (wsOptions.server["log-format"] !== "none") app.use(getLogger());
/* --compress enables compression */
if (wsOptions.server.compress) app.use(compress());
@ -138,22 +138,18 @@ function getLogger(){
/* log using --log-format (if supplied) */
var logFormat = wsOptions.server["log-format"];
if(logFormat) {
if (logFormat === "none"){
// do nothing, no logging required
} else {
if (logFormat === "logstalgia"){
/* customised logger :date token, purely to satisfy Logstalgia. */
morgan.token("date", function(){
var d = new Date();
return (d.getDate() + "/" + d.getUTCMonth() + "/" + d.getFullYear() + ":" + d.toTimeString())
.replace("GMT", "").replace(" (BST)", "");
});
logFormat = "combined";
}
return morgan(logFormat);
if (logFormat === "logstalgia"){
/* customised logger :date token, purely to satisfy Logstalgia. */
morgan.token("date", function(){
var d = new Date();
return (d.getDate() + "/" + d.getUTCMonth() + "/" + d.getFullYear() + ":" + d.toTimeString())
.replace("GMT", "").replace(" (BST)", "");
});
logFormat = "combined";
}
return morgan(logFormat);
/* if no `--log-format` was specified, pipe the default format output
into `log-stats`, which prints statistics to the console */
} else {