プロジェクト

全般

プロフィール

バグ #951

完了

詳細タブのツイート時刻をクリックするとmikutterが暴走する.

Shi rusu さんが7年以上前に追加. 7年以上前に更新.

ステータス:
終了
優先度:
通常
担当者:
対象バージョン:
プラグイン名:
ブランチ:
クラッシュする:

説明

ツイートを開いたときに表示される 詳細タブ 中の ツイート時刻 をクリックすると, mikutterが暴走します.
関連付けをすべて消した状態でもそうなります.


ファイル

toshi_a 初音 さんが7年以上前に更新

暴走って具体的にどんな感じですか?フリーズして動かなくなるとか?

Shi rusu さんが7年以上前に更新

大量にCPUリソースを消費したまま, 一切操作を受け付けなくなります.

toshi_a 初音 さんが7年以上前に更新

  • トラッカー機能 から バグ に変更

なるほど、使える状態ではなさそうですね

あひる 家鴨 さんが7年以上前に更新

developブランチですが、同様の手順でmikutterが落ちたので取れたログを添付します
現象としてはセグって落ちてるように見えます

Akira Ouchi さんが7年以上前に更新

私の環境で確認した場合は以下のエラーとともにクラッシュしました。

/Users/akkie/mikutter_dev/core/mui/gtk_retriever_header_widget.rb:71:in `block in message_opener': undefined local variable or method `retriever' for #<Gtk::RetrieverHeaderWidget:0x7ffb1a310b88 ptr=0x7ffb14e06400>
     from /Users/akkie/mikutter_dev/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `main'
     from /Users/akkie/mikutter_dev/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)

release/3.5ブランチのcommit fd560042e1d698be888363e16fe3380550d844c4です。

コスモ 虹川 さんが7年以上前に更新

dd1ac9aff50d0f667387ee0af12d2e26d6657c43 では通常のツイートでは開く方法が提示されるダイアログがでることになっており、
落ちなくなっているようです。

ただし、システムメッセージではまだ落ちるようです。

/media/Data2/mikutter/core/plugin/display_requirements/display_requirements.rb:181:in `clicked': undefined method `id' for #<Mikutter::System::Message:0x0055fa2bfcc530>
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:160:in `block in event_hooks'
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
/media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit': exit
     from /media/Data2/mikutter/vendor/bundle/ruby/2.3.0/gems/glib2-3.0.8/lib/glib2.rb:41:in `exit_application'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `signal_emit'
     from /media/Data2/mikutter/core/mui/cairo_cell_renderer_message.rb:97:in `block in tree='
     from /media/Data2/mikutter/core/mui/gtk_extension.rb:33:in `block in safety_signal_connect'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `main'
     from /media/Data2/mikutter/core/plugin/gtk/mainloop.rb:10:in `mainloop'
     from mikutter.rb:66:in `boot!'
     from mikutter.rb:95:in `<main>'
notice: mikutter.rb:72:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter.rb:74:in `rescue in boot!': => `SystemExit'
notice: mikutter.rb:99:in `rescue in <main>': catch SystemExit
notice: {MIKUTTER_DIR}/core/configloader.rb:32:in `block in <module:ConfigLoader>': configloader: wrote 1 keys (UserConfig::ui_tab_order)

修正がアドホックですがこれで一応システムメッセージの場合も落ちなくなります。

toshi_a 初音 さんが7年以上前に更新

パッチ見ましたが、コスモさんが言ってるのはツイートの詳細じゃなくてMiraclePainterですかね。とすると別の問題?あとで検証します

toshi_a 初音 さんが7年以上前に更新

  • ステータス新規 から レビュー待ち に変更
  • 担当者Shi rusu にセット
  • 進捗率0 から 100 に変更

報告されたバグも確かにあるんですが、ここの正しい挙動は他のIntentを開く、vanillaのmikutterでの動作でいうと単にブラウザでそのツイートを開くという挙動が正しいので、実装しました。release/3.5ブランチです。
一応そちらでも確認してもらえますか。

日も迫ってるので、 Akira Ouchi も確認してみてもらって、どちらかから報告貰えたらこれでOKとします。

Shi rusu さんが7年以上前に更新

  • ステータスレビュー待ち から 解決 に変更

私の環境で試したところ正常に動作するようになりました.

Akira Ouchi さんが7年以上前に更新

ブラウザが開けることを確認できました

toshi_a 初音 さんが7年以上前に更新

  • ステータス解決 から 終了 に変更

他の形式にエクスポート: Atom PDF