Skip to content

Commit e2dffdb

Browse files
authored
Merge pull request #226 from rcowsill/fix/225-research-dos
Fix crash when research page server-side request fails
2 parents 812bfd7 + 4a4d1db commit e2dffdb

1 file changed

Lines changed: 10 additions & 7 deletions

File tree

app/routes/research.js

Lines changed: 10 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
const ResearchDAO = require("../data/research-dao").ResearchDAO;
2-
const needle = require('needle');
2+
const needle = require("needle");
33
const {
44
environmentalScripts
55
} = require("../../config/config");
@@ -13,14 +13,17 @@ function ResearchHandler(db) {
1313

1414
if (req.query.symbol) {
1515
const url = req.query.url + req.query.symbol;
16-
return needle.get(url, (error, newResponse) => {
17-
if (!error && newResponse.statusCode == 200)
16+
return needle.get(url, (error, newResponse, body) => {
17+
if (!error && newResponse.statusCode === 200) {
1818
res.writeHead(200, {
19-
'Content-Type': 'text/html'
19+
"Content-Type": "text/html"
2020
});
21-
res.write('<h1>The following is the stock information you requested.</h1>\n\n');
22-
res.write('\n\n');
23-
res.write(newResponse.body);
21+
}
22+
res.write("<h1>The following is the stock information you requested.</h1>\n\n");
23+
res.write("\n\n");
24+
if (body) {
25+
res.write(body);
26+
}
2427
return res.end();
2528
});
2629
}

0 commit comments

Comments
 (0)