Behaviour of ObservableMixin when its `off` method is called while its `fire` method is currently iterating over listeners
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
ObservableMixin の fire と off の実装を見つけ、報告にある 2 つのリスナーの例で問題を再現します。dispatch 中にリスナーが自身を削除するケースをカバーする回帰テストを追加します。完了条件は、残りのリスナーが引き続き呼び出されることです。
索引モデルが issue の本文から書いたものです。
説明
Suppose we have two listeners on "close" event.
import txaio
from autobahn.util import ObservableMixin
txaio.use_twisted()
class Foo(ObservableMixin):
def __init__(self):
self.set_valid_events(["someEvent"])
def doSomething(self):
self.fire("someEvent", self)
def listener1(foo):
f.off("someEvent", listener1)
print("listener1 called")
def listener2(foo):
print("listener2 called")
f = Foo()
f.on("someEvent", listener1)
f.on("someEvent", listener2)
f.doSomething()
This will output:
listener1 called
This means, when ObservableMixin off method is called inside listener1,
except for last listeners, then the next listeners (listener2) will never get called.
I think this is because ObservableMixin off method is called while
ObservableMixin fire method is iterating over current listeners of event..
- 主要言語
- Python
- スター
- 2.5k
- フォーク
- 769
- 平均マージ
- 1日 11時間
- マージ済み PR(30日)
- 16
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
crossbario/autobahn-python のほかの issue
-
CI-CD enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
crossbario/autobahn-python#1945 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug CI-CD
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
crossbario/autobahn-python#1932 ·
メンテナーはふだん 1 日以内に返信
-
CI-CD enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
crossbario/autobahn-python#1829 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 4/5 3〜5日 初心者へのやさしさ 22/100
crossbario/autobahn-python#1985 ·
メンテナーはふだん 1 日以内に返信
-
難易度 5/5 1週間以上 初心者へのやさしさ 25/100
crossbario/autobahn-python#1963 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
crossbario/autobahn-python の issue をすべて見る
似ている issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
common-workflow-language/cwl-utils#477 ·
メンテナーはふだん 1 日以内に返信
-
Add dependabotオープンfeat
難易度 2/5 1時間未満 初心者へのやさしさ 72/100
open-nudge/opentemplate#204 ·
メンテナーはふだん 1 日以内に返信
-
github-stale-ci-pr-closer: KV state is double-wrapped, never persists, and hits the depth-32 limit対応中かも @VascoSch92 が今日担当しました。 オープンpriority:medium ready-for-dev
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
OpenHands/extensions#781 ·
メンテナーはふだん 1 日以内に返信
-
bug good first issue
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 1 日以内に返信
-
curriculum documentation quality
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
githubnext/gh-aw-workshop#4638 ·
メンテナーはふだん 1 日以内に返信