TheThing/node-bunyan-lite
1
0
Fork 0
Code Issues Pull requests Projects Releases 1 Wiki Activity

node-bunyan#252 Fixed typo and removed duplicate newline chars

Browse source 
* "client_res" wasn't removed properly resulting in "client_res={}" logs in extras
* duplicate new line characters were added when client_req.address wasn't set
This commit is contained in:
Thomas Heymann 2015-05-14 22:23:51 +01:00 committed by Trent Mick
parent 5660501664
commit 18e6023dd0
1 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
bin/bunyan
View file

@ -880,20 +880,19 @@ function emitRecord(rec, line, opts, stylize) {
var hostHeaderLine = ''; var hostHeaderLine = '';
var s = ''; var s = '';
if (client_req.address) { if (client_req.address) {
hostHeaderLine = 'Host: ' + client_req.address; hostHeaderLine = '\nHost: ' + client_req.address;
if (client_req.port) if (client_req.port)
hostHeaderLine += ':' + client_req.port; hostHeaderLine += ':' + client_req.port;
hostHeaderLine += '\n';
} }
delete client_req.headers; delete client_req.headers;
delete client_req.address; delete client_req.address;
delete client_req.port; delete client_req.port;
s += format('%s %s HTTP/%s\n%s%s', client_req.method, s += format('%s %s HTTP/%s%s%s', client_req.method,
client_req.url, client_req.url,
client_req.httpVersion || '1.1', client_req.httpVersion || '1.1',
hostHeaderLine, hostHeaderLine,
(headers ? (headers ?
Object.keys(headers).map( '\n' + Object.keys(headers).map(
function (h) { function (h) {
return h + ': ' + headers[h]; return h + ': ' + headers[h];
}).join('\n') : }).join('\n') :
@ -972,7 +971,7 @@ function emitRecord(rec, line, opts, stylize) {
} }
if (rec.client_res && typeof (rec.client_res) === 'object') { if (rec.client_res && typeof (rec.client_res) === 'object') {
_res(rec.client_res); _res(rec.client_res);
delete rec.res; delete rec.client_res;
} }
if (rec.err && rec.err.stack) { if (rec.err && rec.err.stack) {