Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Raft] Pending logs events no longer being handled #1156

Closed
chris-j-h opened this issue Mar 26, 2021 · 0 comments · Fixed by #1157
Closed

[Raft] Pending logs events no longer being handled #1156

chris-j-h opened this issue Mar 26, 2021 · 0 comments · Fixed by #1157

Comments

@chris-j-h
Copy link
Collaborator

System information

Current master branch

 ~/Work/quorum-examples/examples/7nodes % geth version
Geth
Version: 1.9.20-stable
Git Commit: eb8f6bf522629bac2eba2e39474fa169753ca712
Git Commit Date: 20210326
Quorum Version: 21.1.0

Issue

#1121 introduced ethereum/go-ethereum#20312 which switches from event.TypeMux to an event.Feed subscription for sending pending logs events to the filters package.

raft.minter::firePendingBlockEvents still sends these events on the event.TypeMux and as a result the filters package is not handling them.

Steps to reproduce the behaviour

This can be observed by subscribing to logs events on a websockets-enabled leader node using something like wscat, e.g.:

~/Work/quorum-examples/examples/7nodes % wscat -c ws://localhost:23004
Connected (press CTRL+C to quit)

> {"id":1, "method":"eth_subscribe", "params":["logs", {"FromBlock":"pending","ToBlock":"pending"}]}

When sending transactions that emit logs, no events are returned by the subscription.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant