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

「メッセージを送信」を選択した時の投稿フォームに最初から入っているメンションにインスタンスの情報が含まれていない #11716

Closed
Hexirp opened this issue Aug 13, 2023 · 0 comments · Fixed by #11721
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR

Comments

@Hexirp
Copy link
Contributor

Hexirp commented Aug 13, 2023

💡 Summary

「メッセージを送信」を選択した時の投稿フォームのデフォルトテキストの中にあるメンションにインスタンスの情報が含まれていません。

os.post({ specified: user, initialText: `@${user.username} ` });

この部分が原因であると私は推測しています。

const canonical = user.host === null ? `@${user.username}` : `@${user.username}@${user.host}`;
os.post({ specified: user, initialText: `${canonical} ` });

このように変更すれば修正できると私は考えています。

(自分用メモ: 同じインスタンスの時は user.host が null となる。メンションでは Punycode にしたものを使うので toUnicode は不要である。ホストが国際化ドメイン名を使っている例は https://misskey.io/@north@ꩰ.com がある。)

🥰 Expected Behavior

あるインスタンス A にログインしているとする。他のインスタンス B のユーザー X のページのメニューから「メッセージを送信」を選択したり、他のインスタンスのユーザー X の投稿のメニューから「ユーザー」→「メッセージを送信」を選択したりすると、投稿フォームが開き、そのフォームは最初から X へのダイレクト投稿となっており、その本文には既に X へのメンションが入っている。

🤬 Actual Behavior

あるインスタンス A にログインしているとする。他のインスタンス B のユーザー X のページのメニューから「メッセージを送信」を選択したり、他のインスタンスのユーザー X の投稿のメニューから「ユーザー」→「メッセージを送信」を選択したりすると、投稿フォームが開き、そのフォームは最初から X へのダイレクト投稿となっており、その本文には X へのメンションが入っているが、そのメンションにはインスタンス B の情報が欠落している。そのため、メンションをいちいち修正する必要があるのに加えて、意図しないユーザーにメンションを送信してしまう危険性が高まってしまう。

📝 Steps to Reproduce

  1. misskey.io インスタンスにいるとする。
  2. https://misskey.io/@[email protected] を閲覧し、右上のメニューから「メッセージを送信」をクリックする。
  3. 出てきた投稿フォームには @[email protected] ではなく @ipc_bot というテキストが入っている。

📌 Environment

💻 Frontend

  • Model and OS of the device(s): Windows 11 Home 22H2
  • Browser: Chrome 115.0.5790.171 (Official Build) (64 bit)
  • Server URL: https://misskey.io/about
  • Misskey: 13.14.2-io.4
@Hexirp Hexirp added the ⚠️bug? This might be a bug label Aug 13, 2023
@acid-chicken acid-chicken added 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR and removed ⚠️bug? This might be a bug labels Aug 14, 2023
Hexirp added a commit to Hexirp/misskey that referenced this issue Aug 14, 2023
あるサーバー A にいるとする。他のサーバー B のユーザー X へ
「メッセージを送信」しようとしたとする。その時に出てくる投稿
フォームには X へのメンションが最初から入っている。

しかし、そのメンションには B の情報が入っておらず、 A の
同名ユーザー X へのメンションとなってしまっている。

See misskey-dev#11716
syuilo pushed a commit that referenced this issue Aug 20, 2023
* fix(frontend): "メッセージを送信" の初期テキストを

あるサーバー A にいるとする。他のサーバー B のユーザー X へ
「メッセージを送信」しようとしたとする。その時に出てくる投稿
フォームには X へのメンションが最初から入っている。

しかし、そのメンションには B の情報が入っておらず、 A の
同名ユーザー X へのメンションとなってしまっている。

See #11716

* Update CHANGELOG.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants