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

Summaly proxy利用時にプレイヤーが動作しないことがある #13194

Closed
1 task done
mei23 opened this issue Feb 7, 2024 · 0 comments · Fixed by #13196
Closed
1 task done

Summaly proxy利用時にプレイヤーが動作しないことがある #13194

mei23 opened this issue Feb 7, 2024 · 0 comments · Fixed by #13196
Labels
⚠️bug? This might be a bug

Comments

@mei23
Copy link
Contributor

mei23 commented Feb 7, 2024

💡 Summary

#10306 以降、URLプレビューの埋め込みプレイヤーでallowプロパティを要求するようになっている。
ただし、外部のSummaly proxy利用時にはallowプロパティが存在しないことがあるため、その場合はプレイヤーが動作しない。
また、内部Summalyにはallowのバリデーションが入っているが、Summaly Proxy利用時にはallowをそのまま使用してしまうため、Misskey側でもバリデーションが必要そう。

🥰 Expected Behavior

  1. allowプロパティがなくてもプレイヤーが表示できる
  2. Summalyで拒否しているpermissonは入らない

🤬 Actual Behavior

  1. 表示できない
  2. Summalyで拒否しているpermissionもallowに含まれてしまう

📝 Steps to Reproduce

  1. allow属性を出力しないSummaly proxyを設定する

💻 Frontend Environment

* Model and OS of the device(s):
* Browser:
* Server URL:
* Misskey:develop

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:develop
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
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
1 participant