Skip to content

Commit 2233c78

Browse files
committedAug 10, 2013
Merge branch 'use-request-jsdom' into master
2 parents 7e0e4cf + a1f7432 commit 2233c78

File tree

2 files changed

+23
-4
lines changed

2 files changed

+23
-4
lines changed
 

‎index.js

+20-3
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
var urlparser = require("url-parser");
2+
var request = require("request");
3+
var jsdom = require("jsdom");
24

35
module.exports.test = function test(url) {
46
parsedurl = urlparser.parse(url);
@@ -18,8 +20,23 @@ module.exports.test = function test(url) {
1820
return false;
1921
};
2022

21-
// TODO: this should be split into multiple methods
2223
module.exports.download = function download(url, options, callback) {
23-
// TODO: this needs to be implemented
24-
throw new Error("not implemented");
24+
request.get(url, function requestcallback(error, response, body) {
25+
var metadata = null;
26+
27+
if (!error && response.statusCode == 200) {
28+
var window = jsdom.jsdom(body).createWindow();
29+
var title = window.document.getElementsByTagName("title")[0].innerHTML;
30+
31+
metadata = {
32+
"html": {
33+
"title": title,
34+
},
35+
};
36+
}
37+
38+
callback(error, metadata);
39+
});
40+
41+
return undefined;
2542
};

‎package.json

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
{
22
"name": "papermonk-downloader-plosone",
33
"description": "plosone.org scraper",
4-
"version": "0.0.1",
4+
"version": "0.0.2",
55
"readmeFilename": "README.md",
66
"homepage": "https://github.com/kanzure/papermonk-downloader-plosone",
77
"author": {
@@ -43,6 +43,8 @@
4343
},
4444
"dependencies": {
4545
"url-parser": ">0",
46+
"request": "~2.26.0",
47+
"jsdom": "~0.8.0",
4648
"tape": ">0"
4749
},
4850
"optionalDependencies": {

0 commit comments

Comments
 (0)