Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Images from URLs dont Always Embed #31

Open
ibrokemypie opened this issue Jan 11, 2019 · 8 comments
Open

Images from URLs dont Always Embed #31

ibrokemypie opened this issue Jan 11, 2019 · 8 comments
Assignees

Comments

@ibrokemypie
Copy link

ibrokemypie commented Jan 11, 2019

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.

@tamaina
Copy link

tamaina commented Feb 17, 2022

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

扱いが分からんけど

@4ioskd
Copy link

4ioskd commented Feb 17, 2022

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

@tamaina
Copy link

tamaina commented Feb 17, 2022

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

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

@4ioskd
Copy link

4ioskd commented Feb 17, 2022

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

@tamaina
Copy link

tamaina commented Feb 17, 2022

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

@KisaragiEffective
Copy link

KisaragiEffective commented Jun 13, 2024

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
Copy link

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

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

@anatawa12
Copy link
Member

anatawa12 commented Jul 18, 2024

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

@anatawa12 anatawa12 transferred this issue from misskey-dev/misskey Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants