misskey-dev / summaly

🔍 Get a summary of any web page
MIT License
22 stars 10 forks source link

Images from URLs dont Always Embed #31

Open ibrokemypie opened 5 years ago

ibrokemypie commented 5 years ago

Summary

Sometimes when you create a post containing a link directly to an image, the image is embedded in the post, but other times it is not embeded

Expected Behavior

Embed linked image

Actual Behavior

Shows the normal link preview

Steps to Reproduce

  1. Create a post containing https://imgs.xkcd.com/comics/what_if.jpg

Environment

Iridium browser, macOS

--- Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/68568217-images-from-urls-dont-always-embed?utm_campaign=plugin&utm_content=tracker%2F51027576&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F51027576&utm_medium=issues&utm_source=github).
tamaina commented 2 years ago

これはCloseしなくてもいいと思う

扱いが分からんけど

4ioskd commented 2 years ago

おそらくペーストしたURL先のサービスが、画像を埋め込むためのメタ情報等を提供していたり、しなかったりするからだと思う。こういったサービス全てに対応するのは、あまりにも非生産的なように思える。

tamaina commented 2 years ago

「URL先がpngやjpegなどの画像」である場合についてはsummalyマターで、メタ情報云々の問題ではない・summaly自体の処理は簡単

ただ、UIをどう作るかが難しいところだと思う

4ioskd commented 2 years ago

なるほど。。 そもそも、ノート(投稿)に現状の機能以上の拡張性をもたらすかどうかは、議論対象になりそう(こういったissueが他にもありそうなので)

tamaina commented 2 years ago

議論対象というよりはしゅいろが決めることだと思う

KisaragiEffective commented 4 months ago

triage: 現行の認識ではmisskey-dev/summarlyによりOGPが埋め込まれているなら正しくリンクプレビューが表示される。しかし、issue起票者によってあげられたリンクはContent-Typeimage/jpegのため (snippet 1)、このissueのポイントはそのようなbare-imageでもリンクのプレビューとして表示するかどうかになると思われる。

snippet 1:

$ curl https://imgs.xkcd.com/comics/what_if.jpg --head
HTTP/1.1 200 OK
Connection: keep-alive
Content-Length: 98556
Server: nginx
Content-Type: image/jpeg
Last-Modified: Sun, 04 Jan 2009 02:24:45 GMT
ETag: "49601ded-180fc"
Expires: Thu, 13 Jun 2024 18:07:28 GMT
Cache-Control: max-age=300
Accept-Ranges: bytes
Date: Thu, 13 Jun 2024 18:03:51 GMT
Via: 1.1 varnish
Age: 82
X-Served-By: cache-itm1220056-ITM
X-Cache: HIT
X-Cache-Hits: 1
X-Timer: S1718301831.175991,VS0,VE1
KisaragiEffective commented 4 months ago

議論対象というよりはしゅいろが決めることだと思う

@syuilo とのことなのでご判断をお願いします 🙏🏻

anatawa12 commented 3 months ago

少なくとも現在は仕様であるし、summaly側の問題なのでtransferします