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

名前を空白文字列だけにできるように戻す #218

Closed
Sayamame-beans opened this issue Jul 31, 2024 · 10 comments · Fixed by #219
Closed

名前を空白文字列だけにできるように戻す #218

Sayamame-beans opened this issue Jul 31, 2024 · 10 comments · Fixed by #219
Labels
✨Feature This adds/improves/enhances a feature

Comments

@Sayamame-beans
Copy link

Sayamame-beans commented Jul 31, 2024

概要

misskey-dev#14119 (表示名の設定が空欄の場合、アカウント名(私の場合Sayabeans)が表示されるようになる)をrevertする

目的

名前への空白文字の入力を防ぐ理由が特に無い
また、防いだとしてもその他の空白文字や:blank:があるので効果が無い
misskey-dev#14088 (comment) (←この問題はこの問題の側を解決するべき)

@Sayamame-beans Sayamame-beans added the ✨Feature This adds/improves/enhances a feature label Jul 31, 2024
@Sayamame-beans
Copy link
Author

@anatawa12 今回のリリース(2024.7.0-kinel.1?)に含めます

@anatawa12
Copy link
Collaborator

うーん:blank:で効果がないはそうなんだけど、わざわざ許可する意味を見いだせない

@Sayamame-beans
Copy link
Author

Sayamame-beans commented Aug 1, 2024

うーん:blank:で効果がないはそうなんだけど、わざわざ許可する意味を見いだせない

にりみすは既にこれに該当するユーザーが居るので、逆に、わざわざ禁止する意味を見いだせなければ弾きたい(入れたくない)という気持ちです

@anatawa12
Copy link
Collaborator

私からしたらupstreamから変える意味がないなら変えたくない気持ちがある

@anatawa12
Copy link
Collaborator

@niwaniwa りらさんに判断してもらおう

@niwaniwa
Copy link
Collaborator

niwaniwa commented Aug 3, 2024

さやまめさんのおっしゃる通り、特段禁止する必要は感じないのかなと思います(現状使用しているユーザーも存在するため。)
私の意見としてはユーザー表現の手段を抑制するような変更は、セキュリティ的なデメリットが存在しない場合upstreamに追従しなくても良いと思います。
(Upstreamからわざわざ変更するのもな、、、というのもわかる)

@anatawa12
Copy link
Collaborator

となるとトリム自体も消す感じですかね

@Sayamame-beans
Copy link
Author

うーん
空白だけの長い名前によって、アカウント名やサーバーURLを隠せてしまうというのがありますが、これは空白でなくても発生する問題ではあり…
特段空白だけを規制する必要はない(トリムもrevertする)か?という気持ちと、空文字列("")で表示名を空に出来たら十分か?という気持ちと…(にりみすに居る方は多分アカウント名を隠すような長さにはしていなかった気がするので)

@anatawa12
Copy link
Collaborator

PRまとまってるので両方revertがらくではある

@Sayamame-beans
Copy link
Author

じゃあ両方revertで良いかな〜
空白文字列固有で、かつ、この部分で対応することが適切な問題で困った時に導入を考えようかなと思います

@anatawa12 anatawa12 mentioned this issue Aug 3, 2024
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants