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.

Sign up for GitHub

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

Jump to bottom

全てのサイトに対応するのではなく、ドメインを絞って対応する #75

Open
ncaq opened this issue Feb 4, 2023 · 2 comments
Open

全てのサイトに対応するのではなく、ドメインを絞って対応する #75

ncaq opened this issue Feb 4, 2023 · 2 comments
Labels
Priority: Low Type: Feature New Feature Type: Refactoring A code change that neither fixes a bug nor adds a feature

Comments

@ncaq
Copy link
Owner

ncaq commented Feb 4, 2023

などとは逆行する方向性なのですが、
やはり全てのサイトで動くアドオンというのはセキュリティ的に危ないし、
ユーザから見ても怪しい。
現状無意味に動くことが多いのでパフォーマンス的にも問題がある。
phpbbは特にいろんなサイトが独自にホスティングしているのでサポートしたいところですが、
少なくとも対応を始めるまではパーミッションを絞ったほうが良さそうである。
@ncaq ncaq added Type: Feature New Feature Priority: Low Type: Refactoring A code change that neither fixes a bug nor adds a feature Estimate: 4 labels Feb 4, 2023
@ncaq ncaq mentioned this issue Feb 4, 2023
Open
@ncaq
Copy link
Owner Author

ncaq commented Feb 4, 2023

Discourse: 対応 · Issue #24 · ncaq/goodbye-rfc-2822-date-time
対応にも問題があるな。

@ncaq
Copy link
Owner Author

ncaq commented Aug 5, 2023

GitHub Enterpriseをホスティングしてる人にとってはホスト名弄れたほうが良いかもしれません。
確か権限って動的に追加できるんでしたっけ?

@ncaq ncaq removed the Estimate: 4 label Oct 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Priority: Low Type: Feature New Feature Type: Refactoring A code change that neither fixes a bug nor adds a feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ncaq