ブラウザでURLを開く機能の改善

[sanomari]

PR の目的

「ブラウザで（URLを）開く」機能を改善します。

カテゴリ

• 仕様変更
• 不具合修正

PR の背景

#1702 セキュリティ問題を起こしやすいShellExecute関数を直接呼ばないようにリファクタリングする
　↓
#1705 クリッカブルURL機能から実行可能ファイルを直接実行できるのは仕様かどうか
　↓
関連付けられたプログラム（ブラウザ）で開くという機能定義から見て
仕様ではなさそうなので、不具合として対処します。

PR のメリット

URLが含まれるテキストの編集中に意図せずプログラムを実行してしまう事態を防止できます。

PR のデメリット (トレードオフとかあれば)

従来の挙動を便利に活用していた人が困るかも知れません。

仕様・動作説明

変更点1、ブラウズ機能から OpenWithBrowser を呼び出す際にローカルファイルパスをそのまま渡していたのを、「file:///」プレフィックスを付けて渡すように変更します。
変更点2、OpenWithBrowserのパラメータ仕様を変更します。変更前はURLとUNCパス、ローカルのフルパスを受け付けていましたが、変更後はURLのみを受け付けるようにします。また、従来は指定されたURLをそのままシェルに渡していましたが、変更後はWindowsのURL解析機能を呼び出してURLスキームをチェックし、「file」プロトコルを開くときに「http」プロトコルに関連付けられたプログラムを使うように変更します。

PR の影響範囲

• ファイル→ブラウズでローカルスクリプトを実行することができなくなります。
  ※従来はbatやcmdの編集中に「ブラウズ」でスクリプト実行できていました。（これも不具合）
  ※ブラウザ上で動くjavascriptの実行確認は可能です。
• クリッカブルURLのダブルクリックでexeファイルを実行することができなくなります。
  ※httpプロトコルに関連付けられたプログラムでexeを開くと対象ファイルが「ダウンロード」されます。
  ※表示するつもりでダウンロードファイルが増えると悲しいので、exeは失敗扱いにします。

関連 issue, PR

resolve #1705

参考資料

https://www.glamenv-septzen.net/view/695#ide5cdab
https://newoldthing.wordpress.com/2007/03/23/how-does-your-browsers-know-that-its-not-the-default-browser/

[sonarcloud]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication

[AppVeyorBot]

✅ Build sakura 1.0.3870 completed (commit 3afb07da32 by @sanomari)

[kengoide]

私は賛成です。

従来の挙動を便利に活用していた人が困るかも知れません。

一応は「外部コマンド実行」で代替可能です。従前同様のコマンドが不可欠ならツール以下に別に設置したほうが良いでしょう。

[sanomari]

ブラウザでURLを開く機能の改善

私は賛成です。

従来の挙動を便利に活用していた人が困るかも知れません。

一応は「外部コマンド実行」で代替可能です。従前同様のコマンドが不可欠ならツール以下に別に設置したほうが良いでしょう。

ご賛同ありがとうございます。

懸念しているのは、何かのチェック用マクロに組み込んで便利に使っている人の存在です。

ツール配下に既存互換のメニューを入れる対応をとる場合、メインメニューの仕様が障害になりそうです。
組み込みのメインメニューは、開発者が簡単には変えられないような細工が入っています。