Skip to content

Commit 431e401

Browse files
committed
Feedback from code review. Improving logging of the domain names that will be ignored.
1 parent d9d655c commit 431e401

File tree

1 file changed

+1
-3
lines changed

1 file changed

+1
-3
lines changed

ElectronNET.Host/main.js

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -84,9 +84,7 @@ if (manifestJsonFile.ignoreAllCertificateErrors) {
8484
// Bypass SSL/TLS certificate errors only for the domain names specified in the electron.manifest.json file.
8585
if (manifestJsonFile.hasOwnProperty('domainNamesToIgnoreCertificateErrors')) {
8686
if (manifestJsonFile.domainNamesToIgnoreCertificateErrors.length > 0) {
87-
manifestJsonFile.domainNamesToIgnoreCertificateErrors.forEach(function (site) {
88-
console.log('SSL/TLS certificate errors will be ignored for ' + site);
89-
});
87+
console.log(`SSL/TLS certificate errors will be ignored for ${manifestJsonFile.domainNamesToIgnoreCertificateErrors.join(', ')}`);
9088

9189
app.on('certificate-error', (event, webContents, url, error, certificate, callback) => {
9290
if (shouldIgnoreCertificateForUrl(url)) {

0 commit comments

Comments
 (0)