Merge pull request #10713 from electron/remote-node-warning-2

Tiny fix: String comparison for remote node warning
This commit is contained in:
Felix Rieseberg 2017-10-07 10:04:49 -07:00 committed by GitHub
commit 77d4927e8b

View file

@ -129,7 +129,9 @@ if (nodeIntegration === 'true') {
}
}
if (/(https?)|(ftp):/.test(window.location.protocol)) {
if (window.location.protocol === 'https:' ||
window.location.protocol === 'http:' ||
window.location.protocol === 'ftp:') {
let warning = 'This renderer process has Node.js integration enabled '
warning += 'and attempted to load remote content. This exposes users of this app to severe '
warning += 'security risks.\n'