From b81f3b70c6310f739d2a6a2c23435e7f6e235b05 Mon Sep 17 00:00:00 2001 From: Nate Silva Date: Fri, 2 Nov 2012 16:04:58 -0700 Subject: [PATCH] log the IP address as seen by Express, if available MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit For `remote-addr`, the logging middleware should use `req.ip`, the IP address seen by Express. This is especially useful when behind a proxy with the Express “trust proxy” option. If `req.ip` is not enabled, fallback to the traditional method. --- lib/middleware/logger.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lib/middleware/logger.js b/lib/middleware/logger.js index d29958f4f..fb8753481 100644 --- a/lib/middleware/logger.js +++ b/lib/middleware/logger.js @@ -299,7 +299,7 @@ exports.token('referrer', function(req){ */ exports.token('remote-addr', function(req){ - return req.socket && (req.socket.remoteAddress || (req.socket.socket && req.socket.socket.remoteAddress)); + return req.ip || req.socket && (req.socket.remoteAddress || (req.socket.socket && req.socket.socket.remoteAddress)); }); /**