follow-redirects 0.0.6 → 0.0.7

package/README.md

@@ -10,7 +10,7 @@ Drop in replacement for Nodes `http` and `https` that automatically follows redi
 
 [![NPM](https://nodei.co/npm/follow-redirects.png?downloads=true)](https://nodei.co/npm/follow-redirects/)
 
-`follow-redirects` provides [request](https://nodejs.org/api/http.html#http_http_request_options_callback) and [get](https://nodejs.org/api/http.html#http_http_get_options_callback) 
+`follow-redirects` provides [request](https://nodejs.org/api/http.html#http_http_request_options_callback) and [get](https://nodejs.org/api/http.html#http_http_get_options_callback)
  methods that behave identically to those found on the native [http](https://nodejs.org/api/http.html#http_http_request_options_callback) and [https](https://nodejs.org/api/https.html#https_https_request_options_callback)
  modules, with the exception that they will seamlessly follow redirects.
 
@@ -48,7 +48,7 @@ https.request({
   host: 'bitly.com',
   path: '/UHfDGO',
 }, function (res) {
-  console.log(res.fetchedUrls); 
+  console.log(res.fetchedUrls);
   // [ 'http://duckduckgo.com/robots.txt',  'http://bitly.com/UHfDGO' ]
 });
 ```
@@ -56,7 +56,7 @@ https.request({
 ## Browserify Usage
 
 Due to the way `XMLHttpRequest` works, the `browserify` versions of `http` and `https` already follow redirects.
- If you are *only* targetting the browser, then this library has little value for you. If you want to write cross 
+ If you are *only* targetting the browser, then this library has little value for you. If you want to write cross
  platform code for node and the browser, `follow-redirects` provides a great solution for making the native node
  modules behave the same as they do in browserified builds in the browser. To avoid bundling unnecessary code
  you should tell browserify to swap out `follow-redirects` with the standard modules when bundling.
@@ -78,10 +78,10 @@ You can then replace `follow-redirects` in your browserify configuration like so
 
 The `browserify-http` module has not kept pace with node development, and no long behaves identically to the native
  module when running in the browser. If you are experiencing problems, you may want to check out
- [browserify-http-2](https://www.npmjs.com/package/http-browserify-2). It is more actively maintained and 
+ [browserify-http-2](https://www.npmjs.com/package/http-browserify-2). It is more actively maintained and
  attempts to address a few of the shortcomings of `browserify-http`. In that case, your browserify config should
  look something like this:
- 
+
 ```javascript
 "browser": {
   "follow-redirects/http"  : "browserify-http-2/http",
@@ -94,7 +94,7 @@ The `browserify-http` module has not kept pace with node development, and no lon
 Pull Requests are always welcome. Please [file an issue](https://github.com/olalonde/follow-redirects/issues)
  detailing your proposal before you invest your valuable time. Additional features and bug fixes should be accompanied
  by tests. You can run the test suite locally with a simple `npm test` command.
- 
+
 ## Debug Logging
 
 `follow-redirects` uses the excellent [debug](https://www.npmjs.com/package/debug) for logging. To turn on logging

package/create.js

@@ -2,6 +2,7 @@
 var url = require('url');
 var debug = require('debug')('follow-redirects');
 var assert = require('assert');
+var consume = require('stream-consume');
 
 module.exports = function(_nativeProtocols) {
   var nativeProtocols = {};
@@ -40,7 +41,7 @@ module.exports = function(_nativeProtocols) {
 
         // we are going to follow the redirect, but in node 0.10 we must first attach a data listener
         // to consume the stream and send the 'end' event
-        res.on('data', function() {});
+        consume(res);
 
         // need to use url.resolve() in case location is a relative URL
         var redirectUrl = url.resolve(fetchedUrl, res.headers.location);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "follow-redirects",
-  "version": "0.0.6",
+  "version": "0.0.7",
   "description": "HTTP and HTTPS modules that follow redirects.",
   "main": "index.js",
   "scripts": {
@@ -42,7 +42,8 @@
     "https.js"
   ],
   "dependencies": {
-    "debug": "^2.2.0"
+    "debug": "^2.2.0",
+    "stream-consume": "^0.1.0"
   },
   "devDependencies": {
     "bluebird": "^2.9.30",