Project

General

Profile

バグ #1206

userstream で取得するツイートの表示が extended_tweet 形式になっていない

Added by Izumi Tsutsui over 1 year ago. Updated over 1 year ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
ブランチ:
topic/1206-extended-tweet-in-user-stream
クラッシュする:
No

Description

#921#note-7 で記載した件ですが、「別のチケットで進めたほうがよい」とのことなので書きました。

問題

  • UserStream で取得された extended_tweet が compat tweet の体裁で表示されてしまう
  • REST や FilterStream で取得する extended_tweet は意図通り表示される
  • 最初にツイートを取得した状態でキャッシュに入るので、 UserStream で取得されたツイートは他のタブでもずっと compat 体裁で表示される
  • 一度 compat 体裁で表示されたツイートも、再起動後には REST 等で再取得されるので extended での表示になる

修正トライ

  • 4/14の突発てオフでのとしぁさんとの会話の中で
    「リクエストのほうが足りてないとかじゃないか」
    という話があったので調べてみたところ、
    以下の userstream リクエストの修正で userstream でも extended_tweet は正常に取れるようになりました。
    attachment:api_shortcuts.rb.diff
修正前後の mikutter を同時に起動して別マシンからテスト投稿した時の表示は以下:

過去の経緯?

https://social.mikutter.hachune.net/@toshi_a/99861507043910666
でとしぁさんが挙げられている
source:core/plugin/twitter/mikutwitter/api_call_support.rb@50429700#L118
では

 # Streaming APIにはtweet_modeスイッチが効かないとかTwitterアホか!?

と書かれていますが、少なくとも今現在は上記の通り tweet_mode=extended_tweet は効いているようです。

その他

  • mikutterにおける自分自身の投稿ツイート表示についてのとしぁさんコメント
    https://social.mikutter.hachune.net/@toshi_a/99861627822711844
    @tsutsuii 自分がツイートしたものを別のmikutterで見るというのが一番簡単でしょうね。Twitterはツイート作成APIエンドポイント(/statuses/update.json)の戻り値として、作成されたツイートを返してくるのですが、これはREST APIなので互換モード(Extended Tweetつき)形式になっていて、mikutterは全てのレスポンスをTL表示に利用するため検証になりません(たまに投稿APIのレスポンスよりUserStreamのほうが早く返してくることがあるため、CompatになったりExtendedになったりすると思われる)。
    投稿APIをmikutterから叩かなければ回避できるので、別のクライアントか、もう一つmikutterを立ち上げて投稿すると惑わされることがないと思います

Files

Associated revisions

Revision f4b654d8 (diff)
Added by Izumi Tsutsui over 1 year ago

userstream で取得するツイートの表示が extended_tweet 形式になっていない refs #1206

History

#1

Updated by Akira Ouchi over 1 year ago

#921-7 のパッチでextended_tweetが展開されていることを私のほうでも確認できました。

余談ですが、開いたり閉じたりを繰り返したせいかmikutterのCK/CSで
ストリーミングAPIに接続できなくなり、適当な自前のCK/CSに差し替えてテストしました…

#2

Updated by toshi_a 初音 over 1 year ago

  • Status changed from 分類待ち to パッチ適用待ち
#3

Updated by toshi_a 初音 over 1 year ago

  • Status changed from パッチ適用待ち to レビュー待ち
  • Assignee set to Izumi Tsutsui
  • ブランチ set to topic/1206-extended-tweet-in-user-stream

凍結されたので何も分からんですが、あっきぃも確認してくれたということなのでパッチ当てました。
そのまま当てましたが、一応確認お願いします。

#4

Updated by Izumi Tsutsui over 1 year ago

  • Status changed from レビュー待ち to マージ待ち
  • Assignee changed from Izumi Tsutsui to toshi_a 初音

topic/1206-extended-tweet-in-user-stream と 3.6.5 とで
それぞれのそれっぽいツイートを受信してみて直っていると思います。
https://twitter.com/tsutsuii/status/986630694532071424

てオフ駆動で直ってよかったです(゜∀。)

#5

Updated by toshi_a 初音 over 1 year ago

  • Status changed from マージ待ち to 終了

mergeしました

Also available in: Atom PDF