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

feat: add oEmbed support #6

Merged
merged 15 commits into from
Mar 13, 2023
Merged

feat: add oEmbed support #6

merged 15 commits into from
Mar 13, 2023

Conversation

saschanaz
Copy link
Member

misskey-dev/misskey#10009 のため

httpsなどの制限はこっちでしました(その方がテストしやすいし他のユーザーにも安全そうでしたので)

@tamaina
Copy link

tamaina commented Mar 11, 2023

README.mdへ仕様も書いていただけると…

@tamaina
Copy link

tamaina commented Mar 11, 2023

oEmbedを新たに追加するのではなくplayerにallowを追加して済ませるのはだめ?

@saschanaz
Copy link
Member Author

えっ、天才?

@tamaina
Copy link

tamaina commented Mar 11, 2023

既存実装にsandboxがないのまずい?

@saschanaz
Copy link
Member Author

既存実装にsandboxがないのまずい?

あ、そうかもです

@saschanaz
Copy link
Member Author

でもog:videoのiframeでも何でもできるのでどんなにまずいのかを聞くとちょっと微妙

@tamaina
Copy link

tamaina commented Mar 11, 2023

でもog:videoのiframeでも何でもできる

現状でも不味かった

allow追加の方向で

@saschanaz
Copy link
Member Author

ツイッターはog:videoどうやってるのか見に行きましたがいつの間にか閉止されていた

@saschanaz
Copy link
Member Author

その場合og:videoのallowはどうなりますか?[]は微妙ですのでMisskey仕様と同様にautoplay;encrypted-media;fullscreenに?

@saschanaz
Copy link
Member Author

nullでいいかも

@tamaina
Copy link

tamaina commented Mar 11, 2023

配列かnullかというのはちょっとコードを書くのが面倒な気がしなくもないけど、空配列というのも違うか…?

@saschanaz
Copy link
Member Author

配列かnullかというのはちょっとコードを書くのが面倒な気がしなくもないけど、空配列というのも違うか…?

何もallowしなくなったら壊れるものがありますので(fullscreenボタンとか、YouTubeだったら再生できなくなる可能性も)

@tamaina
Copy link

tamaina commented Mar 11, 2023

YouTubeはoEmbedに対応していないの

@saschanaz
Copy link
Member Author

oEmbedは対応してますがtype=richじゃなくtype=videoです(機能は同じ)

oEmbedあったらそっちを優先する方が楽かも…?

でもog:videoだけ対応しているページはやっぱり壊れそうですね

@tamaina
Copy link

tamaina commented Mar 11, 2023

oEmbed優先の方向がいいと思う

初期値はencrypted-media;fullscreenあたりでいいのでは(autoplayいる?)

@saschanaz
Copy link
Member Author

oEmbed優先の方向がいいと思う

初期値はencrypted-media;fullscreenあたりでいいのでは(autoplayいる?)

個人的には要らないですしそもそもMisskeyがautopreviewしてないので要らないです

@tamaina tamaina merged commit eab3766 into master Mar 13, 2023
@saschanaz saschanaz deleted the oembed branch March 13, 2023 17:49
@tamaina
Copy link

tamaina commented Mar 13, 2023

👍👍👍👍

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

Successfully merging this pull request may close these issues.

2 participants