Merge pull request #22 from DanielRuf/feature/detailed-certificate-ch…

…ains feature: add detailed parameter to output the certificate chain
johncrisostomo · Oct 18, 2019 · ee1a1fd · ee1a1fd
2 parents c915b14 + f3be86e
commit ee1a1fd
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/index.js b/index.js
@@ -42,9 +42,9 @@ function validateUrl(url) {
   }
 }
 
-function handleRequest(options, resolve, reject) {
+function handleRequest(options, detailed = false, resolve, reject) {
   return https.get(options, function(res) {
-    var certificate = res.socket.getPeerCertificate();
+    var certificate = res.socket.getPeerCertificate(detailed);
 
     if (isEmpty(certificate) || certificate === null) {
       reject({ message: 'The website did not provide a certificate' });
@@ -57,7 +57,7 @@ function handleRequest(options, resolve, reject) {
   });
 }
 
-function get(url, timeout, port, protocol) {
+function get(url, timeout, port, protocol, detailed) {
   validateUrl(url);
 
   port = port || 443;
@@ -66,7 +66,7 @@ function get(url, timeout, port, protocol) {
   var options = getOptions(url, port, protocol);
 
   return new Promise(function(resolve, reject) {
-    var req = handleRequest(options, resolve, reject);
+    var req = handleRequest(options, detailed, resolve, reject);
 
     if (timeout) {
       req.setTimeout(timeout, function() {