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

common-log-format dep no longer directly required

Browse Source
This commit is contained in:
Lloyd Brookes
2014-06-14 15:46:27 +01:00
parent e1ec12a5cf
commit 57672fd114
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
bin/ws.js
View File

@ -14,7 +14,6 @@ var dope = require("console-dope"),
compress = require("compression"),
homePath = require("home-path"),
byteSize = require("byte-size"),
clf = require("common-log-format"),
logStats = require("stream-log-stats");
var usage =
@ -91,10 +90,11 @@ if (argv.config){
}
app.use(morgan(argv["log-format"]));
/* if no specific `--log-format` required, pipe the default web log output
into `log-stats`, which prints statistics to the console */
} else {
var statStream = clf();
statStream.pipe(logStats());
app.use(morgan({ stream: statStream }));
app.use(morgan({ stream: logStats() }));
}
/* --compress enables compression */
@ -115,7 +115,7 @@ if (argv.config){
})
.listen(argv.port);
/* write status to stderr so stdout can be piped to disk ($ ws > log.txt) */
/* write status to stderr (stdout is reserved for web log output) */
if (path.resolve(argv.directory) === process.cwd()){
dope.error("serving at %underline{%s}", "http://localhost:" + argv.port);
} else {