-
Notifications
You must be signed in to change notification settings - Fork 12
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
Conversation
README.mdへ仕様も書いていただけると… |
oEmbedを新たに追加するのではなくplayerにallowを追加して済ませるのはだめ? |
えっ、天才? |
既存実装にsandboxがないのまずい? |
あ、そうかもです |
でもog:videoのiframeでも何でもできるのでどんなにまずいのかを聞くとちょっと微妙 |
現状でも不味かった allow追加の方向で |
ツイッターはog:videoどうやってるのか見に行きましたがいつの間にか閉止されていた |
その場合og:videoのallowはどうなりますか? |
|
配列かnullかというのはちょっとコードを書くのが面倒な気がしなくもないけど、空配列というのも違うか…? |
何もallowしなくなったら壊れるものがありますので(fullscreenボタンとか、YouTubeだったら再生できなくなる可能性も) |
YouTubeはoEmbedに対応していないの |
oEmbedは対応してますがtype=richじゃなくtype=videoです(機能は同じ) oEmbedあったらそっちを優先する方が楽かも…? でもog:videoだけ対応しているページはやっぱり壊れそうですね |
oEmbed優先の方向がいいと思う 初期値はencrypted-media;fullscreenあたりでいいのでは(autoplayいる?) |
個人的には要らないですしそもそもMisskeyがautopreviewしてないので要らないです |
👍👍👍👍 |
misskey-dev/misskey#10009 のため
httpsなどの制限はこっちでしました(その方がテストしやすいし他のユーザーにも安全そうでしたので)