From 431e4014516df296d278e454e7363f42e6058227 Mon Sep 17 00:00:00 2001 From: javierlarota Date: Thu, 9 Dec 2021 04:22:35 +0000 Subject: [PATCH] Feedback from code review. Improving logging of the domain names that will be ignored. --- ElectronNET.Host/main.js | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/ElectronNET.Host/main.js b/ElectronNET.Host/main.js index 2ca46cf..b8367f3 100644 --- a/ElectronNET.Host/main.js +++ b/ElectronNET.Host/main.js @@ -84,9 +84,7 @@ if (manifestJsonFile.ignoreAllCertificateErrors) { // Bypass SSL/TLS certificate errors only for the domain names specified in the electron.manifest.json file. if (manifestJsonFile.hasOwnProperty('domainNamesToIgnoreCertificateErrors')) { if (manifestJsonFile.domainNamesToIgnoreCertificateErrors.length > 0) { - manifestJsonFile.domainNamesToIgnoreCertificateErrors.forEach(function (site) { - console.log('SSL/TLS certificate errors will be ignored for ' + site); - }); + console.log(`SSL/TLS certificate errors will be ignored for ${manifestJsonFile.domainNamesToIgnoreCertificateErrors.join(', ')}`); app.on('certificate-error', (event, webContents, url, error, certificate, callback) => { if (shouldIgnoreCertificateForUrl(url)) {