[fix] Only set x-forward-* headers if req.connection and req.connec…

…tion.socket
http-party · Aug 29, 2011 · de4a6fe · de4a6fe
1 parent 340be42
commit de4a6fe
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/node-http-proxy/http-proxy.js b/lib/node-http-proxy/http-proxy.js
@@ -126,10 +126,10 @@ HttpProxy.prototype.proxyRequest = function (req, res, buffer) {
   // * `x-forwarded-proto`: Protocol of the original request
   // * `x-forwarded-port`: Port of the original request.
   //
-  if (this.enable.xforward) {
+  if (this.enable.xforward && req.connection && req.connection.socket) {
     req.headers['x-forwarded-for']   = req.connection.remoteAddress || req.connection.socket.remoteAddress;
     req.headers['x-forwarded-port']  = req.connection.remotePort || req.connection.socket.remotePort;
-    req.headers['x-forwarded-proto'] = res.connection.pair ? 'https' : 'http';
+    req.headers['x-forwarded-proto'] = req.connection.pair ? 'https' : 'http';
   }
 
   //