maxmustermann/kibana
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

Merge pull request #2932 from simianhacker/fix/request_logger

Browse source 
Adding a check to ensure res._header is set before checking values
This commit is contained in:
Joe Fleming 2015-02-09 14:12:10 -07:00
parent 0639ea9425 5982268679
commit a760235bab
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/server/lib/requestLogger.js
View file

@ -7,9 +7,8 @@ module.exports = function (options) {
var _req = _.pick(req, ['url', 'headers', 'method']);
_req.connection = _.pick(req.connection, ['remoteAddress', 'remotePort']);
res.end = function (chunk, encoding) {
var contentLength = parseInt(res._header['content-length'], 10);
res.contentLength = parseInt(res.getHeader('content-length') || 0, 10);
res.responseTime = (new Date()).getTime() - startTime.getTime();
res.contentLength = isNaN(contentLength) ? 0 : contentLength;
end.call(res, chunk, encoding);
logger.info({ req: _req, res: res }, '%s %s %d - %dms', req.method, req.url, res.statusCode, res.responseTime);
};