summaly icon indicating copy to clipboard operation
summaly copied to clipboard

Images from URLs dont Always Embed

Open ibrokemypie opened this issue 6 years ago • 8 comments

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! We accept bounties via Bountysource.

ibrokemypie avatar Jan 11 '19 13:01 ibrokemypie

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

扱いが分からんけど

tamaina avatar Feb 17 '22 14:02 tamaina

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

4ioskd avatar Feb 17 '22 14:02 4ioskd

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

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

tamaina avatar Feb 17 '22 14:02 tamaina

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

4ioskd avatar Feb 17 '22 14:02 4ioskd

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

tamaina avatar Feb 17 '22 14:02 tamaina

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 avatar Jun 13 '24 18:06 KisaragiEffective

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

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

KisaragiEffective avatar Jun 13 '24 18:06 KisaragiEffective

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

anatawa12 avatar Jul 18 '24 06:07 anatawa12