Merge pull request #4959 from atom/check-url-before-opening

Only open http/https links in new-window example
This commit is contained in:
Kevin Sawicki
2016-03-31 15:41:33 -07:00

View File

@@ -606,7 +606,10 @@ The following example code opens the new url in system's default browser.
```javascript
webview.addEventListener('new-window', function(e) {
require('electron').shell.openExternal(e.url);
var protocol = require('url').parse(e.url).protocol;
if (protocol === 'http:' || protocol === 'https:') {
require('electron').shell.openExternal(e.url);
}
});
```