satorumurmur / bibi

Bibi | EPUB Reader on your website.
https://bibi.epub.link
MIT License
843 stars 123 forks source link

Fix for Windows.Web.IUriToStreamResolver #79

Closed kurema closed 4 years ago

kurema commented 4 years ago

WebView.NavigateToLocalStreamUriで利用する際に、上記部分の修正が必要でした。 これはUWP(Universal Windows Platform)でローカル生成したウェブコンテンツを表示する機能で、URLは"ms-local-stream://"で始まります。(利用元)。

HTML版を普通のアプリ内蔵ビュワーとして使うことは考えられますし、その場合スキーム名がhttps?//ではない事も多いはずです。

一々ホワイトリストで追加するのもおかしいですし、

typeof BookData == 'string' ? (/^data:/.test(BookData) ? 'Base64' : 'URI') :

で問題ないならその方が良いと思います。 こうなっているのは何か理由があると考えて、UWP向けの修正だけを行っています。

satorumurmur commented 4 years ago

@kurema ありがとうございます。

自分がパターンを見極め切れていないかぎりはなるべくホワイトリストで、という実装方針が出ていた箇所ですが、おっしゃるとおり、ここについてはコメント内でご提案くださったコードのほうが良いと私も思います。

ひとまずはプルリクエストどおりマージさせていただきつつ、次のリリースバージョンでは、コメントでご提案いただいた方針を採用するつもりです。