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

Twitchのプレイヤーをiframe埋め込みで開けない #10759

Closed
tamaina opened this issue May 4, 2023 · 13 comments · Fixed by #14247
Closed

Twitchのプレイヤーをiframe埋め込みで開けない #10759

tamaina opened this issue May 4, 2023 · 13 comments · Fixed by #14247
Labels
⚠️bug? This might be a bug

Comments

@tamaina
Copy link
Contributor

tamaina commented May 4, 2023

summaly関連?

@tamaina tamaina added the ⚠️bug? This might be a bug label May 4, 2023
@tamaina
Copy link
Contributor Author

tamaina commented May 4, 2023

{
	"title": "magronachan",
	"icon": "https://p1.a9z.dev/proxy/preview.webp?url=https%3A%2F%2Fstatic.twitchcdn.net%2Fassets%2Ffavicon-32-e29e246c157142c94346.png&preview=1",
	"description": "余",
	"thumbnail": "https://p1.a9z.dev/proxy/preview.webp?url=https%3A%2F%2Fstatic-cdn.jtvnw.net%2Fjtv_user_pictures%2F2ff09c5a-d673-4260-9723-a74b389a8859-profile_image-300x300.png&preview=1",
	"player": {
		"url": "https://player.twitch.tv/?channel=magronachan&player=facebook&autoplay=true&parent=meta.tag",
		"width": 620,
		"height": 378,
		"allow": [
			"autoplay",
			"encrypted-media",
			"fullscreen"
		]
	},
	"sitename": "Twitch",
	"sensitive": false,
	"url": "https://www.twitch.tv/magronachan"
}

image

@tamaina
Copy link
Contributor Author

tamaina commented May 4, 2023

https://support.mozilla.org/ja/kb/xframe-neterror-page?as=u&utm_source=inproduct

X-Frame-OptionsかCSPが悪いらしい

@tamaina
Copy link
Contributor Author

tamaina commented May 25, 2023

今試したらplayerが消えてた

@tamaina tamaina closed this as completed May 25, 2023
@acid-chicken acid-chicken closed this as not planned Won't fix, can't repro, duplicate, stale May 25, 2023
@mei23
Copy link
Contributor

mei23 commented May 25, 2023

Twitchがフレーム埋め込みをいわゆる許可リスト制にしちゃってるからかも

Content-Security-Policy: 
frame-ancestors
https://reddit.com
https://www.reddit.com
https://old.reddit.com
https://new.reddit.com
https://www.redditmedia.com
https://twitter.com
https://cards-frame.twitter.com
https://tweetdeck.twitter.com
https://discordapp.com
https://discord.com
https://embedly.com
https://cdn.embedly.com
https://facebook.com
https://www.facebook.com
https://vk.com

微妙に日本ではマイナーなSNSも入ってるから、みんなでリクエストすれば追加してくれるのかしら…

@acid-chicken
Copy link
Member

微妙に日本ではマイナーなSNSも入ってるから、みんなでリクエストすれば追加してくれるのかしら…

オリジンが別だから回避用の共通オリジンを持つとかしないと無理そう

@mei23
Copy link
Contributor

mei23 commented May 25, 2023

https://dev.twitch.tv/docs/embed/video-and-clips/
埋め込みURLを parent=example.com みたいにしたらCSPに入れてもらえた
Content-Security-Policy: frame-ancestors https://example.com

なお、ogの生成だとparent=meta.tagというダミーっぽいのが設定されてて出てこない。

@acid-chicken
Copy link
Member

なら summaly のプラグインとして追加すれば良さそう

@mei23
Copy link
Contributor

mei23 commented May 25, 2023

summalyの応答はインスタンス間で共用されることがあるからそこに入れたくないかも。
Misskeyがsummalyの応答受け取ってPlayer呼ぶときに書き換えたほうがいいかも。

@acid-chicken
Copy link
Member

あー

@acid-chicken
Copy link
Member

今試したらplayerが消えてた

あとこれは単純に Twitch のバグくさい

@mei23
Copy link
Contributor

mei23 commented May 25, 2023

Misskeyがsummalyの応答受け取ってPlayer呼ぶときに書き換えたほうがいいかも。

それぞれのMisskeyサーバーがクライアントに返すタイミング
つまり Misskey backend の /url endportで書き換えるのが良さそう

@acid-chicken acid-chicken reopened this May 25, 2023
@tamaina
Copy link
Contributor Author

tamaina commented May 26, 2023

summaly proxy廃止しようぜというのがよぎった

@acid-chicken
Copy link
Member

summaly proxy廃止しようぜというのがよぎった

ターゲットウェブサイトのスパイクは軽視すべきではない

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants