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

サーバーサイレンス/ブロックのサーバー指定方法を改善して欲しい(サブドメインを含むか否か) #14330

Open
1 task
Sayamame-beans opened this issue Jul 30, 2024 · 0 comments
Labels
[Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features. ✨Feature This adds/improves/enhances a feature 👮Moderation Issues to monitor server users and content and maintain public safety.

Comments

@Sayamame-beans
Copy link
Member

Sayamame-beans commented Jul 30, 2024

Summary

#9263 により、サーバーブロックにはendswithによる判定が追加され、サブドメインが自動的に対象として含まれるようになっているようです。(ただし、サーバー一覧での表示にはこの処理が含まれていないことから、 #12293 でfederation/instancesの挙動修正が試みられています)
しかしながら、(明示的な指定をしなくとも)特定サーバーをブロックすると関連するサブドメイン付きのサーバーもブロックされてしまうのは意図しない挙動にもなり得ると思います。

また、サブドメインもブロックされるという旨は、 #12031 でのサーバーサイレンス追加時に削除されているため、サブドメインが判定に含まれていることは認識されづらい状態となっていました。

そこで、指定内容にワイルドカードがある場合はサブドメインを含む、無い場合はそのドメインだけにする、というような判定をして欲しいと思います。(例: *.example.comexample.comかの2種類の入力を受け付ける)
これにより、意図した場合のみサブドメインを対象に含めることが出来ます。(もちろん、従来の挙動との互換性を加味し、「デフォルトでは*を含むようにし、オプション等で*を含まないのをデフォルトに出来るようにする。いずれの場合も手動編集で挙動を変えられる。」とするのも良いと思います。)

備考: #13842 の判定処理ではendswithを含んでいないため、サブドメインにはメディアサイレンスが作用しません。(従って12293の問題も発生しません)

Purpose

より直感的な挙動と利便性強化

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@Sayamame-beans Sayamame-beans added the ✨Feature This adds/improves/enhances a feature label Jul 30, 2024
@samunohito samunohito added the 👮Moderation Issues to monitor server users and content and maintain public safety. label Nov 27, 2024
@samunohito samunohito added the [Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features. label Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Feat] ControlPanel Issues related to existing functionality, such as bugs or adding small features. ✨Feature This adds/improves/enhances a feature 👮Moderation Issues to monitor server users and content and maintain public safety.
Projects
Status: No status
Status: No status
Development

No branches or pull requests

2 participants