Differential D15463

[Chronik] Improve logs in ws.rs
ClosedPublic
Actions

Authored by tobias_ruck on Feb 15 2024, 23:11.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABC0fdf6b22d4f8: [Chronik] Improve logs in ws.rs

Summary

This generally just looks better but also helps with D15461.

Depends on D15462.

Test Plan

run regtest, subscribe and observe logs

Diff Detail

Repository

rABC Bitcoin ABC

Branch

chronik-improve-ws-logs

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 27170
Build 53906: Build Diff	build-chronik-plugins · chronik-client-integration-tests · build-chronik
Build 53905: arc lint + arc unit

Event Timeline

tobias_ruck created this revision.Feb 15 2024, 23:11

Herald added a reviewer: Restricted Project. · View Herald TranscriptFeb 15 2024, 23:11

tobias_ruck requested review of this revision.Feb 15 2024, 23:11

Harbormaster completed remote builds in B27170: Diff 45283.Feb 15 2024, 23:31

Fabien accepted this revision.Feb 16 2024, 09:26

This revision is now accepted and ready to land.Feb 16 2024, 09:26

Closed by commit rABC0fdf6b22d4f8: [Chronik] Improve logs in ws.rs (authored by tobias_ruck). · Explain WhyFeb 16 2024, 09:27

This revision was automatically updated to reflect the committed changes.

tobias_ruck added a commit: rABC0fdf6b22d4f8: [Chronik] Improve logs in ws.rs.

Fabien mentioned this in D15461: [chronik] Fix websocket subscription in tests.Feb 16 2024, 09:48

Revision Contents
Changeset List

Path

Size

chronik/

chronik-http/

src/

ws.rs

8 lines

Diff 45283

View Options

chronik/chronik-http/src/ws.rs

[Chronik] Improve logs in ws.rsClosedPublicActions