[Chronik] Add `WsSubTxs` subscription
ClosedPublic
Actions

Authored by tobias_ruck on Sun, Nov 23, 00:01.

Details

Reviewers

Fabien
alitayin

Group Reviewers

Restricted Project

Commits

rABC8aafa64105bd: [Chronik] Add `WsSubTxs` subscription

Summary

WebSocket subscriptions to simply receive all transactions. This is useful for a block explorer.

Test Plan

./test/functional/test_runner.py chronik_ws

Repository

rABC Bitcoin ABC

Branch

chronik-sub-to-all-txs

Lint

Lint Passed

Severity	Location	Code	Message
Auto-Fix	test/functional/chronik_ws.py:1	RUFF-CHECK	Code style violation

Unit

No Test Coverage

Build Status

Buildable 35155
Build 69768: Build Diff	electrum-functional-tests · chronik-client-integration-tests · ecash-lib-integration-tests · build-without-wallet · build-clang · build-diff · build-clang-tidy · ecash-agora-integration-tests · build-chronik · build-debug · build-chronik-plugins
Build 69767: arc lint + arc unit

@bot guix-linux

rename "AllTxs" to just "Txs", add assert_recv_all_any_order, and finish chronik_ws test

fix lint

@bot guix-linux

tobias_ruck retitled this revision from [Chronik] Add `WsAllTxs` subscription to [Chronik] Add `WsSublTxs` subscription.Mon, Nov 24, 18:58

perfect. gogo

This revision is now accepted and ready to land.Mon, Nov 24, 19:06

Fabien added inline comments.

chronik/chronik-indexer/src/subs.rs
56 ↗	(On Diff #56745)	At some point we should make this configurable, but out of scope for this diff
test/functional/chronik_ws.py
644 ↗	(On Diff #56745)	please lower to 2s, not need to spend 10s proving nothing happens