stream-http
Advanced tools
Comparing version 1.0.2 to 1.1.1
// var Base64 = require('Base64') | ||
var capability = require('./capability') | ||
var foreach = require('foreach') | ||
var indexOf = require('indexof') | ||
var inherits = require('inherits') | ||
var keys = require('object-keys') | ||
var response = require('./response') | ||
var stream = require('stream') | ||
var inherits = require('inherits') | ||
@@ -66,3 +67,10 @@ var IncomingMessage = response.IncomingMessage | ||
var self = this | ||
self._headers[name.toLowerCase()] = { | ||
var lowerName = name.toLowerCase() | ||
// This check is not necessary, but it prevents warnings from browsers about setting unsafe | ||
// headers. To be honest I'm not entirely sure hiding these warnings is a good thing, but | ||
// http-browserify did it, so I will too. | ||
if (indexOf(unsafeHeaders, lowerName) !== -1) | ||
return | ||
self._headers[lowerName] = { | ||
name: name, | ||
@@ -113,3 +121,3 @@ value: value | ||
mode: 'cors', | ||
credentials: opts.credentials ? 'include' : 'omit' | ||
credentials: opts.withCredentials ? 'include' : 'omit' | ||
}).then(function (response) { | ||
@@ -239,1 +247,26 @@ self._fetchResponse = response | ||
ClientRequest.prototype.setSocketKeepAlive = function () {} | ||
// Taken from http://www.w3.org/TR/XMLHttpRequest/#the-setrequestheader%28%29-method | ||
var unsafeHeaders = [ | ||
'accept-charset', | ||
'accept-encoding', | ||
'access-control-request-headers', | ||
'access-control-request-method', | ||
'connection', | ||
'content-length', | ||
'cookie', | ||
'cookie2', | ||
'date', | ||
'dnt', | ||
'expect', | ||
'host', | ||
'keep-alive', | ||
'origin', | ||
'referer', | ||
'te', | ||
'trailer', | ||
'transfer-encoding', | ||
'upgrade', | ||
'user-agent', | ||
'via' | ||
]; |
var capability = require('./capability') | ||
var foreach = require('foreach') | ||
var inherits = require('inherits') | ||
var stream = require('stream') | ||
var inherits = require('inherits') | ||
@@ -42,3 +42,2 @@ var rStates = exports.readyStates = { | ||
self.push(null) | ||
self.emit('close') | ||
return | ||
@@ -155,3 +154,2 @@ } | ||
self.push(null) | ||
self.emit('close') | ||
} | ||
@@ -166,4 +164,3 @@ // reader.onerror = ??? // TODO: this | ||
self.push(null) | ||
self.emit('close') | ||
} | ||
} |
{ | ||
"name": "stream-http", | ||
"version": "1.0.2", | ||
"version": "1.1.1", | ||
"description": "Streaming http in the browser", | ||
@@ -21,2 +21,3 @@ "main": "index.js", | ||
"foreach": "^2.0.5", | ||
"indexof": "0.0.1", | ||
"inherits": "^2.0.1", | ||
@@ -23,0 +24,0 @@ "object-keys": "1.0.4", |
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
License Policy Violation
LicenseThis package is not allowed per your license policy. Review the package's license to ensure compliance.
Found 1 instance in 1 package
65933
874
6
+ Addedindexof@0.0.1
+ Addedindexof@0.0.1(transitive)