ユーザー名が正しく表示されない

[blhsrwznrghfzpr]

現象

インスタンス名が https://hogedon.com のときに、トゥートID or URLに https://hogedon.com/web/statuses/012345678901234567 の形式で toot を取得すると、
その toot が他所のインスタンスの toot でも、
ユーザー名が @[email protected] に固定されてしまっています。

原因

https://github.com/hidao80/mastogetter/blob/4750bd7571272b8292050be20323bf118aa8c234/js/common.js#L96
にて、入力されたインスタンス名を表示にしようしていますが、
おそらく API レスポンスから正しい値を取得する必要があります。

[blhsrwznrghfzpr]

インスタンス qiitadon.com から他所のインスタンスの toot を取ると
account.acct が [email protected] のようになってて使えそうだけど、
今度は qiitadon.com インスタンスの account.acct が username のみになってしまう…

[hidao80]

@blhsrwznrghfzpr 仕様です（つよき）
現段階では、「トゥートID or URL」テキストボックスではインスタンスを判別していません。

「トゥートID or URL」テキストボックス利用時は左隣の「インスタンス名」欄に別途インスタンスのURLを入力してください。
「インスタンス名」テキストボックスのデフォルト値は「https://qiitadon.com」です。

【二つのインスタンスからトゥートをまとめている例】

改善案、プルリク募集中です。#37 も見てね。

[hidao80]

@blhsrwznrghfzpr あ゛あ゛あ゛そういうことか！
まとめ表示画面で全てのユーザのインスタンスがuserid@qiitadonになるということですね！
それは純然たるバグです。直しましょう。

[yumetodo]

MastodonのTLから来ました。

個人的にはaccount.acctを見るよりはurlからインスタンス名引っ張るほうがいいんじゃないかという気がしています。

[hidao80]

と思ったけど、FTLに流れているものは当該インスタンスに保存されているので、「userid@当該インスタンス」が付くのが正常な動作です。

ぜんぜんわからない。俺は雰囲気で mastodon をやっている。

[yumetodo]

@hidao80 それは自分の所属インスタンス視点での話であって、mastogetterとしてどうするかは別問題でしょう

[blhsrwznrghfzpr]

とりあえずURLなどからインスタンス名を引っ張る方針でPR考えてみます

[hidao80]

@yumetodo 現在、mastogetter ではまとめ情報を管理する URL の get パラメータに1つしかインスタンス URL を持つことができません。これは現段階の仕様です。

たしかに、mastogetter としてどうするかは別問題ですが、解決方法の検討もされていないのが現状です。
本 Issue で解決したい問題だと考えています。

[KEINOS]

んんん？？ @[email protected] ？？

そのアカウントあるけど、そこからトゥートしとらんのですが、どうなってんの？！

と思って検証してみたら、そういうことか。理解。

Mastodon のバージョン互換もありそうだから、力技を使う方向なのかしら。となるとリクエスト数が多くなりそうな気配。

[yumetodo]

https://github.com/tootsuite/mastodon/blob/a6269b2f83e3eed1a8ab545f5756cd7b582075f5/spec/helpers/accounts_helper_spec.rb#L57-L65

とおもったんだけど、acctについてのテスト見てたらちょっと首かしげてます。(見てるところが違う・・・？

[yumetodo]

えっ、ここをくっつけようって話なんですがリクエスト数とか

現在、mastogetter ではまとめ情報を管理する URL の get パラメータに1つしかインスタンス URL を持つことができません。これは現段階の仕様です。

ってなんか話がすれ違っている・・・？

[yumetodo]

      let acct = account.get('acct');

      if (acct.indexOf('@') === -1 && localDomain) {
        acct = `${acct}@${localDomain}`;
      }

      displayName = <bdi><strong className='display-name__html' dangerouslySetInnerHTML={{ __html: account.get('display_name_html') }} /></bdi>;
      suffix      = <span className='display-name__account'>@{acct}</span>;

https://github.com/tootsuite/mastodon/blob/59fd622adc18cb094146f7cda1025782d505404e/app/javascript/mastodon/components/display_name.js#L73-L81

あっ

[yumetodo]

https://github.com/fedibird/mastodon/blame/b8514561394767a10d3cf40132ada24d938c1680/spec/helpers/statuses_helper_spec.rb
みてて気がついたけど

mastodon/mastodon#8011
で仕様変わった・・・？

[KEINOS]

https://paiza.io/projects/BJFF9fHVUIyP6SL1xNromA @ paiza.IO

上記で、複数インスタンスの実際の API のレスポンスを見たところ、account の acct url と display_name を使うのが良い気がする。

[yumetodo]

あーびっくりした、acctって言われて。

なんかbreaking changeが割と最近入っている感もあるし

https://mastodon.cardina1.red/@lo48576/103448649823505013
あー表示だから変えてまえ！ で変わったタイプの…… (つらそう)

ActivityPub 自体は @[email protected] 形式のアカウント表示を規定してないから、独自 API に頼るしかないんですね……
それにしても API の破壊的変更はひどい

みたいな指摘もあるので

<div class="box"><a class="display-name" href="${obj.account.url}" target="_blank">${obj.account.display_name}<span>@${obj.account.username}@${(new URL(obj.url)).hostname}</span></a>

みたいな感じでいいのでは。

対応ブラウザ問題なければPR投げますよ

[hidao80]

@yumetodo 明文化はしていませんが、IE と旧 edge は対応外で開発進めているので勧めてくださるとありがたいです！ PR お待ちしています！