Follow redirects (rbren#23)

manuel-di-iorio · rbren · commit 4967c43e5a88 · 2017-03-27T14:13:49.000-04:00
* browserify

* 2.6.0

* Revert package.json, test/ and dist/

* Fix settings object for parseURL when passing an empty object
diff --git a/index.js b/index.js
@@ -164,7 +164,7 @@ var decorateItunes = function decorateItunes(json, channel) {
     }
     json.feed.itunes.owner = owner;
   }
-  
+
   PODCAST_TOP_FIELDS.forEach(function(f) {
     if (channel['itunes:' + f]) json.feed.itunes[f] = channel['itunes:' + f][0];
   });
@@ -197,7 +197,14 @@ Parser.parseString = function(xml, callback) {
   });
 }
 
-Parser.parseURL = function(feedUrl, callback) {
+Parser.parseURL = function(feedUrl, settings, callback) {
+  if (!callback) {
+    callback = settings;
+    settings = {};
+  }
+  settings.__redirectCount = settings.__redirectCount || 0;
+  if (settings.maxRedirects === undefined) settings.maxRedirects = 1;
+
   var xml = '';
   var get = feedUrl.indexOf('https') === 0 ? HTTPS.get : HTTP.get;
   var parsedUrl = url.parse(feedUrl);
@@ -207,7 +214,12 @@ Parser.parseURL = function(feedUrl, callback) {
     path: parsedUrl.path,
     headers: {'User-Agent': 'rss-parser'}
   }, function(res) {
-    if (res.statusCode >= 300) return callback(new Error("Status code " + res.statusCode))
+    if (res.statusCode >= 300 && res.statusCode < 400 && res.headers['location']) {
+      if (settings.maxRedirects === 0) return callback(new Error("Status code " + res.statusCode));
+      if (settings.__redirectCount === settings.maxRedirects) return callback(new Error("Too many redirects"));
+      settings.__redirectCount++;
+      return Parser.parseURL(res.headers['location'], settings, callback);
+    }
     res.setEncoding('utf8');
     res.on('data', function(chunk) {
       xml += chunk;
