consensus: anchor checkpoint fast-add - #103
Conversation
Signed-off-by: Chris Gianelloni <wolf31o2@blinklabs.io>
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughThe block importer now processes imported blocks with normal validation flags instead of fast-add. A regression test constructs a proof-of-work-valid block with an invalid coinbase height and verifies rejection with Possibly related PRs
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
Summary
addblockimports use full consensus validation instead of unconditionally requesting fast-addValidation
make unit unit-race unit-cover build integration hsd-interop lint vuln workers=2go mod verifyin the root module,hnsutil,hnsutil/psbt, andchaincfg/chainhash--versionsmoke tests forlinux/amd64andlinux/arm64The validation used Go 1.26.5, pinned hsd v8.0.0, and a freshly compiled govulncheck v1.6.0.
Summary by cubic
Anchors fast-add to a known checkpoint header and enforces full consensus validation for
addblockimports to prevent bypassing rules with partial header chains.netsync: Enable fast-add only after the checkpoint header is in the best-header view; keep full validation if a peer stops headers before the checkpoint.addblock: Useblockchain.BFNonefor imports instead ofBFFastAdd.Written for commit aab8c57. Summary will update on new commits.
Summary by CodeRabbit
Bug Fixes
Tests