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

ドメイン部を含まないメンションが受信したインスタンスのアカウント宛と勘違いされる #1427

Closed
unarist opened this issue Apr 9, 2018 · 3 comments
Assignees
Labels
🐛Bug Unexpected behavior 💬Discussion Being discussed or needs discussion

Comments

@unarist
Copy link
Contributor

unarist commented Apr 9, 2018

同じインスタンス内のアカウントへのメンションはドメイン部を書かずに @foo と書くことが多いが、例えば misskey.xyzmastodon.social からこれを受けとると、 @foo@misskey.xyz 宛のメンションとして通知が発生してしまう。クライアント側でのリンク化も同様。

@syuilo syuilo added ActivityPub 🐛Bug Unexpected behavior labels Apr 9, 2018
@syuilo
Copy link
Member

syuilo commented Apr 9, 2018

リモートの投稿内のメンションでホスト部分が省略されている場合、暗黙的にその投稿を行ったユーザーのホストが指定されているとみなした方が良さそうですね

@unarist
Copy link
Contributor Author

unarist commented Apr 10, 2018

改めて確認してみて、他所だとホスト部分込みでメンション書いてもHTML上では省略されるのを思いだしました…。

例えばMastodonではActivityPubのMentionタグを使ってこのあたりを判断していて、少なくとも通知に関しては https://www.w3.org/TR/activitystreams-vocabulary/#microsyntaxes で言及があります。

リンク化については、MastodonではMentionタグで指定されたactorのurlプロパティとhrefが一致するリンクを当該メンションとして扱い、WebUIでプロフィールを開くようにしていたのですが、Pleromaはactor.uriの方をhrefにしていたらしく、という話もあってうーん。

c.f. mastodon/mastodon#6839 mastodon/mastodon#6934

@syuilo syuilo added the 💬Discussion Being discussed or needs discussion label May 6, 2018
syuilo added a commit that referenced this issue May 6, 2018
@syuilo syuilo added the wip label May 6, 2018
@syuilo syuilo self-assigned this May 6, 2018
@syuilo syuilo removed the wip label May 7, 2018
@syuilo
Copy link
Member

syuilo commented May 7, 2018

Fixed

@syuilo syuilo closed this as completed May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛Bug Unexpected behavior 💬Discussion Being discussed or needs discussion
Projects
None yet
Development

No branches or pull requests

2 participants