mirrors/electron
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

feat: add location url to "Node.js Integration with Remote Content" warning (#13973)

Browse source 
* docs: fix typo

* feat: add location url to Node+Remote warning
This commit is contained in:
Charles Kerr 2018-08-07 20:40:21 -05:00 committed by GitHub
parent e6e621515f
commit 29f07889e6
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/renderer/security-warnings.js
View file

@ -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? * @returns {boolean} - Is this a remote protocol?
*/ */
@ -137,8 +137,8 @@ module.exports = {
warnAboutNodeWithRemoteContent: () => { warnAboutNodeWithRemoteContent: () => {
if (getIsRemoteProtocol()) { if (getIsRemoteProtocol()) {
const warning = `This renderer process has Node.js integration enabled const warning = `This renderer process has Node.js integration enabled
and attempted to load remote content. This exposes users of this app to and attempted to load remote content from '${window.location}'. This
severe security risks.\n ${moreInformation}` exposes users of this app to severe security risks.\n ${moreInformation}`
console.warn('%cElectron Security Warning (Node.js Integration with Remote Content)', console.warn('%cElectron Security Warning (Node.js Integration with Remote Content)',
'font-weight: bold;', warning) 'font-weight: bold;', warning)