[ecash-herald] Implement chronik-client 0.9.0 with failover proxy
ClosedPublic
Actions

Authored by bytesofman on Nov 28 2023, 23:53.

Details

Reviewers

Fabien

Group Reviewers

Restricted Project

Commits

rABCb2a1a26df9d8: [ecash-herald] Implement chronik-client 0.9.0 with failover proxy

Summary

Upgrade to chronik-client with automated backup servers. Remove startup bailout on failed connection, an artifact of manual deployments and no support for backup chronik.

Test Plan

npm test
node index.js and msgs are broadcast to telegram bot info in local secrets.js

Diff Detail

Repository

rABC Bitcoin ABC

Branch

herald-better-chronik

Lint

Lint Errors

Severity	Location	Code	Message
Error	apps/ecash-herald/src/main.js:18	ESLINT	no-unused-vars

Unit

No Test Coverage

Build Status

Buildable 25762
Build 51104: Build Diff	ecash-herald-tests
Build 51103: arc lint + arc unit

Event Timeline

bytesofman created this revision.Nov 28 2023, 23:53

Herald added a reviewer: Restricted Project. · View Herald TranscriptNov 28 2023, 23:53

bytesofman requested review of this revision.Nov 28 2023, 23:53

Harbormaster completed remote builds in B25762: Diff 43297.Nov 28 2023, 23:53

lint

Harbormaster completed remote builds in B25763: Diff 43298.Nov 28 2023, 23:55

testing locally and broadcasting to a dev channel

bytesofman added inline comments.Nov 29 2023, 00:04

apps/ecash-herald/src/main.js
23 ↗	(On Diff #43298)	Note that chronik with failover proxy will throw an error if it fails to connect to all of the given servers. So, we are still bailing at startup if all servers are unavailable.