aaron/mattermost-desktop
1
0
Fork 0
Code Issues Pull requests Actions 2 Packages Projects Releases Wiki Activity

Fixed an issue where clicking on a file:// protocol path does not open windows explorer since version 3.5.0.

Browse source
This commit is contained in:
lip-d 2018-02-25 10:51:45 +09:00
parent dcc7819dea
commit 267b8780cf
1 changed files with 23 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

42
src/browser/components/MattermostView.jsx
View file

@ -15,6 +15,25 @@ const ErrorView = require('./ErrorView.jsx');
const preloadJS = `file://${remote.app.getAppPath()}/browser/webview/mattermost_bundle.js`;
function extractFileURL(message) {
const matched = message.match(/Not allowed to load local resource:\s*(.+)/);
if (matched) {
return matched[1];
}
return '';
}
function isNetworkDrive(fileURL) {
const u = url.parse(fileURL);
if (u.protocol === 'file:' && u.host) {
// Disallow localhost, 127.0.0.1, ::1.
if (!u.host.match(/^localhost$|^127\.0\.0\.1$|^\[::1\]$/)) {
return true;
}
}
return false;
}
const MattermostView = createReactClass({
propTypes: {
name: PropTypes.string,
@ -156,9 +175,9 @@ const MattermostView = createReactClass({
case 1:
console.warn(message);
break;
case 2:
const fileURL = this.extractFileURL(e.message);
if (this.isNetworkDrive(fileURL)) {
case 2: {
const fileURL = extractFileURL(e.message);
if (isNetworkDrive(fileURL)) {
// Network drive: Should be allowed.
if (!shell.openExternal(decodeURI(fileURL))) {
console.log(`[${this.props.name}] shell.openExternal failed: ${fileURL}`);
@ -168,6 +187,7 @@ const MattermostView = createReactClass({
console.error(message);
}
break;
}
default:
console.log(message);
break;
@ -239,22 +259,6 @@ const MattermostView = createReactClass({
);
},
extractFileURL(message) {
const matched = message.match(/Not allowed to load local resource:\s*(.+)/);
if (matched) {
return matched[1];
}
return '';
},
isNetworkDrive(fileURL) {
const u = url.parse(fileURL);
if (u.protocol === 'file:' && u.host) {
return true;
}
return false;
},
render() {
const errorView = this.state.errorInfo ? (
<ErrorView