coolaj86
/
ssl-root-cas.js
1
0
Fork 0
Code Issues 7 Pull Requests Releases Activity

Added check for the correct content type in addition to the response status code.

This commit is contained in:
Mike Flynn 2016-06-18 16:41:26 -07:00
parent a3bd1607ec
commit 3ef528ae1b
1 changed files with 7 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ca-store-generator.js
View File

@ -200,7 +200,7 @@ function run(filename) {
if (!fs.existsSync(outputPemsDir)) { if (!fs.existsSync(outputPemsDir)) {
fs.mkdirSync(outputPemsDir); fs.mkdirSync(outputPemsDir);
} }
console.info("Loading latest certificates from " + CERTDB_URL); console.info("Loading latest certificates from " + CERTDB_URL);
request.get(CERTDB_URL, function (error, response, body) { request.get(CERTDB_URL, function (error, response, body) {
@ -217,6 +217,12 @@ function run(filename) {
return; return;
} }
if (response.headers['content-type'] !== 'text/plain') {
console.error("Fetching failed with incorrect content type %s", response.headers['content-type']);
reject({ code: 2, error: "Fetching failed with incorrect content type " + response.headers['content-type'] });
return;
}
var lines = body.split("\n") var lines = body.split("\n")
, certs = parseCertData(lines) , certs = parseCertData(lines)
, pemsFile = path.join(outputPemsDir, 'mozilla-certdata.txt') , pemsFile = path.join(outputPemsDir, 'mozilla-certdata.txt')