From ca37ad74367764cca479a1af63bd7491dc79606b Mon Sep 17 00:00:00 2001
From: Ryan Stevens <stevens.ryan@gmail.com>
Date: Tue, 22 May 2012 12:42:35 -0700
Subject: [PATCH] [fix] x-forwarded-proto sets properly The ternary was
 evaluating truthy for "," + req.connection.pair which is always true because
 its always a non empty string.  Wrapped actual condition to properly
 concatenate the product of the ternary.  let me know if you prefer the style 
  (req.connection.pair ? 'https' : 'http')

https://github.com/nodejitsu/node-http-proxy/issues/250
---
 lib/node-http-proxy/http-proxy.js | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/lib/node-http-proxy/http-proxy.js b/lib/node-http-proxy/http-proxy.js
index bd8434a..e4f4ae6 100644
--- a/lib/node-http-proxy/http-proxy.js
+++ b/lib/node-http-proxy/http-proxy.js
@@ -148,7 +148,7 @@ HttpProxy.prototype.proxyRequest = function (req, res, buffer) {
     }
 
     if (req.headers['x-forwarded-proto']){
-      var protoToAppend = "," + req.connection.pair ? 'https' : 'http';
+      var protoToAppend = "," + (req.connection.pair) ? 'https' : 'http';
       req.headers['x-forwarded-proto'] += protoToAppend;
     } 
     else {
@@ -415,7 +415,7 @@ HttpProxy.prototype.proxyWebSocketRequest = function (req, socket, head, buffer)
     }
 
     if (req.headers['x-forwarded-proto']){
-      var protoToAppend = "," + req.connection.pair ? 'wss' : 'ws';
+      var protoToAppend = "," + (req.connection.pair) ? 'wss' : 'ws';
       req.headers['x-forwarded-proto'] += protoToAppend;
     } 
     else {