Skip to content

Commit b0d3cd4

Browse files
gudezhiaetheon
authored andcommitted
use [request] instead of [http.get] to avoid redirect error (#37)
1 parent b5f4636 commit b0d3cd4

File tree

1 file changed

+11
-19
lines changed

1 file changed

+11
-19
lines changed

lib/Downloader.js

Lines changed: 11 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,5 @@
11
var http = require('http'),
2+
request = require('request'),
23
path = require("path"),
34
url = require("url"),
45
fs = require("fs"),
@@ -63,25 +64,16 @@ Downloader.prototype.download = function _download(results, limit, movieFile, on
6364

6465
// get the subtitle
6566
var scope = this;
66-
var request = http.get(subtitleToDownload, function(response) {
67-
68-
// download and extract the subtitle
69-
var gunzip = zlib.createGunzip();
70-
dest = fs.createWriteStream(downloadFileName);
71-
response.pipe(gunzip).pipe(dest);
72-
73-
dest.on("close", function() {
74-
75-
response.unpipe(gunzip);
76-
response.unpipe(dest);
77-
78-
// emit downloaded event
79-
scope.emit.call(scope, "downloaded", { url: subtitleToDownload, file: downloadFileName});
80-
81-
_download.call(scope, results, --limit, movieFile, onCompleted);
82-
});
83-
84-
});
67+
// download and extract the subtitle
68+
dest = fs.createWriteStream(downloadFileName);
69+
gunzip = zlib.createGunzip()
70+
req = request({url:subtitleToDownload, jar:true, followAllRedirects:true})
71+
req.pipe(gunzip).pipe(dest)
72+
dest.on("close", function() {
73+
// emit downloaded event
74+
scope.emit.call(scope, "downloaded", { url: subtitleToDownload, file: downloadFileName});
75+
_download.call(scope, results, --limit, movieFile, onCompleted);
76+
})
8577

8678
};
8779

0 commit comments

Comments
 (0)