Skip to content

feat: poll Proposed events over http#918

Merged
mikhailUshakoff merged 6 commits intomasterfrom
mu/event-polling
Mar 26, 2026
Merged

feat: poll Proposed events over http#918
mikhailUshakoff merged 6 commits intomasterfrom
mu/event-polling

Conversation

@mikhailUshakoff
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds an HTTP/RPC polling fallback for listening to Proposed (and other) contract events when WebSocket log subscriptions are unavailable, improving robustness across different RPC endpoint types.

Changes:

  • Introduces EventListenerConfig and refactors listen_for_event to support WS subscription with fallback to getLogs polling.
  • Updates the batch proposed receiver to use the new config (including reconnect + polling intervals).
  • Bumps workspace crate versions from 1.34.11 to 1.34.12.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 6 comments.

File Description
common/src/utils/event_listener.rs Refactors event listening to prefer WS subscriptions and fall back to polling via get_block_number/get_logs.
common/src/chain_monitor/batch_proposed_receiver.rs Wires the new listener config into the batch proposed receiver and sets reconnect/poll intervals.
Cargo.toml Workspace version bump to 1.34.12.
Cargo.lock Updates lockfile versions for workspace crates to 1.34.12.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread common/src/utils/event_listener.rs
Comment thread common/src/utils/event_listener.rs
Comment thread common/src/utils/event_listener.rs
Comment thread common/src/utils/event_listener.rs
Comment thread common/src/utils/event_listener.rs
Comment thread common/src/chain_monitor/batch_proposed_receiver.rs
Comment thread common/src/utils/event_listener.rs Outdated
Comment thread common/src/utils/event_listener.rs Outdated
mikhailUshakoff and others added 2 commits March 26, 2026 08:21
Co-authored-by: Maciej Skrzypkowski <mskr@gmx.com>
Co-authored-by: Maciej Skrzypkowski <mskr@gmx.com>
@mikhailUshakoff mikhailUshakoff merged commit 5495d7a into master Mar 26, 2026
7 checks passed
@mikhailUshakoff mikhailUshakoff deleted the mu/event-polling branch March 26, 2026 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants