fix: Merge pull request #153 from adefarge/enable_https

[fix] allow https endpoints
pelias · Jul 9, 2018 · 44f4375 · 44f4375
2 parents aeec1e0 + e5d395b
commit 44f4375
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/lib/request_urls.js b/lib/request_urls.js
@@ -1,4 +1,5 @@
 var http = require('http');
+var https = require('https');
 var request = require('request');
 
 var ExponentialBackoff = require( '../lib/ExponentialBackoff');
@@ -28,7 +29,8 @@ function shouldRetryRequest(res) {
 function request_urls(urls, callback) {
   var total_length = urls.length;
   var responses = {};
-  var agent = new http.Agent({keepAlive: true, maxSockets: 1});
+  var httpAgent = new http.Agent({keepAlive: true, maxSockets: 1});
+  var httpsAgent = new https.Agent({keepAlive: true, maxSockets: 1});
   var intervalId;
   var test_interval = new ExponentialBackoff(1, 5, 1, 20000);
   var delay = test_interval.getBackoff();
@@ -40,6 +42,8 @@ function request_urls(urls, callback) {
 
     var url = urls.pop();
 
+    const agent = (url.startsWith('https:')) ? httpsAgent : httpAgent;
+
     var requestOpts = {
       headers: {'user-agent': 'pelias-fuzzy-tester'},
       url: url,