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

handle 'upgrade' in comma-separated connection header

Browse Source 
Firefox sends `keep-alive, upgrade`, not just `upgrade`,
which the proxy incorrectly turned into `close`
This commit is contained in:
MinRK 2014-09-01 21:25:37 -07:00
parent 554f59c518
commit 51eeebef68
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/http-proxy/common.js
View File

@ -53,7 +53,8 @@ common.setupOutgoing = function(outgoing, options, req, forward) {
if (!outgoing.agent) { if (!outgoing.agent) {
outgoing.headers = outgoing.headers || {}; outgoing.headers = outgoing.headers || {};
if (typeof outgoing.headers.connection !== 'string' if (typeof outgoing.headers.connection !== 'string'
|| outgoing.headers.connection.toLowerCase() !== 'upgrade' || !outgoing.headers.connection.toLowerCase().split(',').some(
function (s) { if (s.trim() === 'upgrade') { return true; } })
) { outgoing.headers.connection = 'close'; } ) { outgoing.headers.connection = 'close'; }
} }