menthas icon indicating copy to clipboard operation
menthas copied to clipboard

サムネイル画像の https 対応

Open hi-noguchi opened this issue 6 years ago • 1 comments

Qiita 記事見ました! 素敵なニュースサイトですね!

恐らく既知の問題とは思いましたが、クロールしてきた各サイトの meta image が http プロトコルなせいで足引っ張られてしまっていますね。以下行にあたる部分で、取得した URL が http だった場合には……という例外対応を入れてあげれば基本的には問題なさそうですが、先方が https 対応できていない場合がつらいですね。 とはいえ画像を全部収集するのも ? な感じで、こういう場合どうするのがベストプラクティスなんだろうか、と思ったのでちょっと書かせていただきました。 (最初、強制的に https で取得するようにして、アクセスできなかった場合にのみ手元に保存する……?)

https://github.com/ytanaka-/menthas/blob/16960602565e3d67670f98b4995f0e6687f1c3e6/src/backend/crawler/webpage-client.js#L39-L43

hi-noguchi avatar Sep 12 '18 07:09 hi-noguchi

ありがとうございます!

mixed-contentの問題は既知だったのですが、ご指摘の通りリンク先がhttpsに対応していない場合がありそうなので一旦許容する仕様になっています。 ですがおっしゃられるように一回httpsでチャレンジしてみるのはありかもしれないですね。軽く調べた感じ用意できているサイトも結構あるようなので。

ytanaka- avatar Sep 12 '18 14:09 ytanaka-