archive-gh-me/node-http-proxy
1
0
Fork 0
mirror of https://github.com/http-party/node-http-proxy.git synced 2025-12-08 20:59:18 +00:00
Code Issues Projects Releases Wiki Activity

[fix] http-proxy should not modify the protocol in redirect request for external sites. Fixes #359.

Browse Source
This commit is contained in:
indexzero 2013-03-09 02:35:50 -05:00
parent 3130665d9f
commit 6a278b3dd8
1 changed files with 11 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
lib/node-http-proxy/http-proxy.js
View File

@ -27,6 +27,7 @@
var events = require('events'),
http = require('http'),
util = require('util'),
url = require('url'),
httpProxy = require('../node-http-proxy');
//
@ -122,7 +123,8 @@ HttpProxy.prototype.proxyRequest = function (req, res, buffer) {
var self = this,
errState = false,
outgoing = new(this.target.base),
reverseProxy;
reverseProxy,
location;
//
// Add common proxy headers to the request so that they can
@ -257,6 +259,8 @@ HttpProxy.prototype.proxyRequest = function (req, res, buffer) {
if ((response.statusCode === 301) || (response.statusCode === 302)
&& typeof response.headers.location !== 'undefined') {
location = url.parse(response.headers.location);
if (location.host === req.headers.host) {
if (self.source.https && !self.target.https) {
response.headers.location = response.headers.location.replace(/^http\:/, 'https:');
}
@ -264,6 +268,7 @@ HttpProxy.prototype.proxyRequest = function (req, res, buffer) {
response.headers.location = response.headers.location.replace(/^https\:/, 'http:');
}
}
}
//
// When the `reverseProxy` `response` ends, end the