From 483d24bac0b882d747dcae69d984c55b757b4d52 Mon Sep 17 00:00:00 2001 From: yan Date: Wed, 28 Sep 2016 13:56:31 -0700 Subject: [PATCH] Unignore cert transparency errors Fix https://github.com/brave/browser-laptop/issues/4159 Auditors: @bridiver Test Plan: go to https://www.mint.com, verify no cert error appears --- app/index.js | 8 -------- 1 file changed, 8 deletions(-) diff --git a/app/index.js b/app/index.js index c5182af29a1..5a4984312c9 100644 --- a/app/index.js +++ b/app/index.js @@ -265,14 +265,6 @@ const notifyCertError = (webContents, url, error, cert) => { app.on('ready', () => { let sessionStateSaveInterval = null app.on('certificate-error', (e, webContents, url, error, cert, resourceType, overridable, strictEnforcement, expiredPreviousDecision, cb) => { - // ignore transparency errors for now because we are blocking - // sites that Chrome isn't for some reason (ex: http://www.mint.com) - if (error === 'net::ERR_CERTIFICATE_TRANSPARENCY_REQUIRED') { - e.preventDefault() - cb(true) - return - } - let host = urlParse(url).host if (host && acceptCertDomains[host] === true) { // Ignore the cert error