From bcc3d4630ff2e287e22eac1a19ea18e21b5398fd Mon Sep 17 00:00:00 2001 From: Abrar Ahmed Date: Mon, 11 Jul 2016 16:01:21 -0700 Subject: [PATCH] add req.url so that connect works with restify --- lib/connect-logger.js | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/lib/connect-logger.js b/lib/connect-logger.js index fc543e9..493266d 100755 --- a/lib/connect-logger.js +++ b/lib/connect-logger.js @@ -128,7 +128,7 @@ function assemble_tokens(req, res, custom_tokens) { }; var default_tokens = []; - default_tokens.push({ token: ':url', replacement: req.originalUrl || req.url }); + default_tokens.push({ token: ':url', replacement: getUrl(req) }); default_tokens.push({ token: ':protocol', replacement: req.protocol }); default_tokens.push({ token: ':hostname', replacement: req.hostname }); default_tokens.push({ token: ':method', replacement: req.method }); @@ -177,6 +177,18 @@ function assemble_tokens(req, res, custom_tokens) { return array_unique_tokens(custom_tokens.concat(default_tokens)); } +/** + * Return request url path, + * adding this function prevents the Cyclomatic Complexity for the assemble_tokens function at low, to pass the tests. + * + * @param {IncomingMessage} req + * @return {String} + * @api private + */ + +function getUrl(req){ + return req.originalUrl || req.url +} /** * Return formatted log line. *