Project

General

Profile

バグ #1344

Mastodon Worldでふぁぼられた時、Current Worldがふぁぼられたアカウントでないと落ちる

Added by Shibafu Midorino 6 months ago. Updated 5 months ago.

Status:
終了
Priority:
通常
Target version:
プラグイン名:
mastodon_sse_streaming
ブランチ:
topic/1344-mastodon-favorite-crash
クラッシュする:
Yes

Description

表題の通り、Mastodonのストリーミングでふぁぼ通知を受けた時、Current Worldがふぁぼを受けたアカウントになっていないと落ちてしまうようです。

mikutter 3.9.0-develop (3715d1ca)

NoMethodError undefined method `user_obj' for nil:NilClass
{MIKUTTER_DIR}/core/plugin/mastodon_sse_streaming/mastodon_sse_streaming.rb:233:in `block (2 levels) in <top (required)>'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/listener.rb:25:in `call'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:116:in `block (2 levels) in call_all_listeners'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:115:in `each'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:115:in `block in call_all_listeners'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:114:in `catch'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:114:in `call_all_listeners'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/pluggaloid-1.1.2/lib/pluggaloid/event.rb:42:in `block in call'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/delayer-0.0.2/lib/delayer/procedure.rb:24:in `run'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/delayer-0.0.2/lib/delayer/extend.rb:58:in `run_once'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/delayer-0.0.2/lib/delayer/extend.rb:30:in `run'
{MIKUTTER_DIR}/vendor/bundle/ruby/2.6.0/gems/delayer-0.0.2/lib/delayer.rb:43:in `method_missing'
{MIKUTTER_DIR}/core/plugin/gtk/delayer.rb:10:in `block in boot'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:7:in `main'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:7:in `block in mainloop'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:6:in `loop'
{MIKUTTER_DIR}/core/plugin/gtk/mainloop.rb:6:in `mainloop'
mikutter.rb:69:in `boot!'
mikutter.rb:105:in `<main>'

Associated revisions

Revision 82ae4540 (diff)
Added by toshi_a 初音 5 months ago

Mastodon::Status.from_me?が返す値が間違っている refs #1344

引数を省略した時に全てのWorldを探索する

Revision 8a945232 (diff)
Added by toshi_a 初音 5 months ago

Status#from_me?とAccount#me?の実装を揃える refs #1344

Revision 34019e04 (diff)
Added by toshi_a 初音 5 months ago

Mastodon Worldでふぁぼられた時、Current Worldがふぁぼられたアカウントでないとクラッシュする refs #1344

History

#1

Updated by toshi_a 初音 5 months ago

  • Status changed from 分類待ち to 実装待ち
#2

Updated by toshi_a 初音 5 months ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to Shibafu Midorino
  • プラグイン名 set to mastodon_sse_streaming
  • ブランチ set to topic/1344-mastodon-favorite-crash

from_me_worldは使う必要がなかったので、調査した結果削除しました。
Status#from_me?とAccount#me?のレスポンスには一貫性があったほうが良いので修正しています。

#3

Updated by Shibafu Midorino 5 months ago

  • Status changed from レビュー待ち to マージ待ち
  • 2つのMastodon Worldを用意して、Current Worldではない方をふぁぼる
  • 同じ条件で、Current Worldにしている方をふぁぼる
  • Current World を mpc World にして、Mastodon Worldの適当なトゥートをふぁぼる

を試行しました。クラッシュすることなく動作しています。

#4

Updated by toshi_a 初音 5 months ago

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

merged.

Also available in: Atom PDF