Page MenuHomePhabricator
Differential D16264

[herald] bump finalization wait time
AbandonedPublic
Actions

Authored by bytesofman on Jun 4 2024, 09:13.

Details

Reviewers
Fabien
Group Reviewers
Restricted Project
Summary

Wait 20s instead of 10s for avalanche finalization before sending 'not finalized' notice

Test Plan

npm test

Diff Detail

Repository
rABC Bitcoin ABC
Branch
herald-timing-adjust
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 29197
Build 57930: Build Diffecash-herald-tests
Build 57929: arc lint + arc unit

Event Timeline

bytesofman created this revision.Jun 4 2024, 09:13
Herald added a reviewer: Restricted Project. · View Herald TranscriptJun 4 2024, 09:13
bytesofman published this revision for review.Jun 4 2024, 09:15
Harbormaster completed remote builds in B29197: Diff 48077.Jun 4 2024, 09:15
Fabien requested changes to this revision.Jun 4 2024, 11:17
Fabien added a subscriber: Fabien.

This happened only once and was very likely due to a network issue since the telegram message also failed at the same time. This bump is too premature.

This revision now requires changes to proceed.Jun 4 2024, 11:17
bytesofman abandoned this revision.Jun 4 2024, 13:27

good point

Revision Contents
Changeset List

PathSize
apps/
ecash-herald/
2 lines

Diff 48077

View Options

apps/ecash-herald/config.js

Loading...