From 5aa247a6277b741188992bc17a5a08c0cbbc86cd Mon Sep 17 00:00:00 2001 From: "trop[bot]" Date: Thu, 9 Aug 2018 20:48:36 -0500 Subject: [PATCH] feat: add location url to "Node.js Integration with Remote Content" warning (backport: 3-0-x) (#13976) * docs: fix typo * feat: add location url to Node+Remote warning --- lib/renderer/security-warnings.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/lib/renderer/security-warnings.js b/lib/renderer/security-warnings.js index 033fcdb1c16..41557164ce7 100644 --- a/lib/renderer/security-warnings.js +++ b/lib/renderer/security-warnings.js @@ -45,7 +45,7 @@ const shouldLogSecurityWarnings = function () { } /** - * Check's if the current window is remote. + * Checks if the current window is remote. * * @returns {boolean} - Is this a remote protocol? */ @@ -137,8 +137,8 @@ module.exports = { warnAboutNodeWithRemoteContent: () => { if (getIsRemoteProtocol()) { const warning = `This renderer process has Node.js integration enabled - and attempted to load remote content. This exposes users of this app to - severe security risks.\n ${moreInformation}` + and attempted to load remote content from '${window.location}'. This + exposes users of this app to severe security risks.\n ${moreInformation}` console.warn('%cElectron Security Warning (Node.js Integration with Remote Content)', 'font-weight: bold;', warning)