In local testing, occasionally a ws will hang and fail to connect. In this case, we should try the next ws.
Details
Details
- Reviewers
emack - Group Reviewers
Restricted Project - Commits
- rABC0a5b61cb244e: [chronik-client] Add timeout failure to ws connections
npm test
I have used this while locally testing Cashtab and can confirm the timeout failures will cause failover proxy to use the next ws. We may want to tweak the timeout depending on behavior we see with apps.
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- cc-ws-timeouts
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 28019 Build 55583: Build Diff chronik-client-integration-tests · chronik-client-tests Build 55582: arc lint + arc unit
Event Timeline
Comment Actions
Failed tests logs:
====== Get blocktxs, txs, and history for SLP NFT1 token txs: "before each" hook for "Gets an SLP NFT1 send tx from the mempool".Get blocktxs, txs, and history for SLP NFT1 token txs "before each" hook for "Gets an SLP NFT1 send tx from the mempool" ======
Error: Timeout of 60000ms exceeded. For async tests and hooks, ensure "done()" is called; if returning a Promise, ensure it resolves. (/work/modules/chronik-client/test/integration/token_slp_nft1.ts)
at listOnTimeout (node:internal/timers:573:17)
at processTimers (node:internal/timers:514:7)Each failure log is accessible here:
Get blocktxs, txs, and history for SLP NFT1 token txs: "before each" hook for "Gets an SLP NFT1 send tx from the mempool".Get blocktxs, txs, and history for SLP NFT1 token txs "before each" hook for "Gets an SLP NFT1 send tx from the mempool"
Comment Actions
The build failed due to an unexpected infrastructure outage. The administrators have been notified to investigate. Sorry for the inconvenience.
| modules/chronik-client/src/failoverProxy.ts | ||
|---|---|---|
| 211 ↗ | (On Diff #46411) | since the plan is to tweak this based on app behavior, put the 5000ms in a config file |
| modules/chronik-client/src/failoverProxy.ts | ||
|---|---|---|
| 211 ↗ | (On Diff #46411) | only used in one place. I don't think we want a config file for a dev library like this one. will fix the magic number thing tho. |