[fix] simple fixes #748 #744 #746

Author: jcrugzz

lib/http-proxy/common.js

@@ -157,7 +157,7 @@ common.urlJoin = function() {
   // on every request
 
   // Handle case where there could be multiple ? in the URL.
-  retSegs.concat(lastSegs);
+  retSegs.push.apply(retSegs, lastSegs);
 
   return retSegs.join('?')
 };

test/lib-http-proxy-common-test.js

@@ -203,7 +203,7 @@ describe('lib/http-proxy/common.js', function () {
       expect(outgoing.path).to.eql('/forward/static/path');
     })
 
-    it.only('should not modify the query string', function () {
+    it('should not modify the query string', function () {
       var outgoing = {};
       common.setupOutgoing(outgoing, {
         target: { path: '/forward' },