Project

General

Profile

バグ #1295

twitterプラグインだけ抜いた状態でつけっぱなしにすると落ちる

Added by Kazuki Yagi 8 months ago. Updated 8 months ago.

Status:
終了
Priority:
通常
Assignee:
Target version:
プラグイン名:
ブランチ:
topic/1295-api-request-file-cache-depends-twitter
クラッシュする:
Yes

Description

core/plugin/twitterを抜いた状態にて、長い間つけっぱなしにしておくと突然落ちます。
--debugをつけない状態では何も出なかったため、--debugをつけて放っておいた所、以下の様なログを出力して落ちていました。

error: {MIKUTTER_DIR}/core/serialthread.rb:83:in `rescue in block in new_thread': uninitialized constant MikuTwitter
Did you mean?  Mikutter
from {MIKUTTER_DIR}/core/plugin/api_request_file_cache.rb:7:in `gc'
from {MIKUTTER_DIR}/core/plugin/api_request_file_cache.rb:12:in `block (2 levels) in <top (required)>'
from {MIKUTTER_DIR}/core/serialthread.rb:70:in `block in new_thread'
notice: mikutter/mikutter.rb:75:in `rescue in boot!': catch exception `SystemExit'
notice: mikutter/mikutter.rb:77:in `rescue in boot!': => `SystemExit'
notice: mikutter/mikutter.rb:109:in `rescue in <main>': catch SystemExit

導入していたplugin(general)

mikutter_justsize        sub_parts_client
fcm                      mikutter_sub_parts_image_flex  twemoji
mikutter-uwm-hommage     mikutter_subparts_nsfw         worldon
mikutter_fcmnotify       show_tweet                     worldon_remote_emoji


再現手順

1. core/plugin/twitterを除去
2. core/plugin/search/search.rb からtwitter-hashtag関連の挙動を削除( /core/plugin/search/search.rb#L10-L12 を消す )
3. core/plugin/search/.mikutter.yml から依存関係としてtwitterを除去(worldonプラグインにsearchが依存プラグインとして挙げられていたため)
4. worldonプラグインやら暇をつぶせるプラグインを入れる
2. 起動する
3. しばらく待つ
4. ドーン

Associated revisions

Revision 70d92821 (diff)
Added by cob odo 8 months ago

api_request_file_cacheプラグインが明示的にtwitterプラグインに依存するように修正 refs #1295

History

#1

Updated by Kazuki Yagi 8 months ago

  • Description updated (diff)
#2

Updated by cob odo 8 months ago

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

報告ありがとうございます。これは修正すべきですね。

#3

Updated by cob odo 8 months ago

  • Status changed from 実装待ち to レビュー待ち
  • Assignee set to Kazuki Yagi
  • ブランチ set to topic/1295-api-request-file-cache-depends-twitter

そして修正したものがこちらになります。 topic/1295-api-request-file-cache-depends-twitter ブランチで確認をお願いします。

#4

Updated by Kazuki Yagi 8 months ago

  • Status changed from レビュー待ち to マージ待ち

ひとまず一日ほど起動し続けてみましたが、少なくとも同様のエラーでクラッシュすることなく動き続けてることが確認できましたので、マージ待ちに変更します。
対応ありがとうございます。

#5

Updated by cob odo 8 months ago

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

ありがとうございました。マージしました。

Also available in: Atom PDF