Skip to content

fix(pxe): warn when block header unavailable for proven/finalized events (backport #22050)#22547

Merged
benesjan merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/backport-22050-pxe-warn
Apr 15, 2026
Merged

fix(pxe): warn when block header unavailable for proven/finalized events (backport #22050)#22547
benesjan merged 1 commit into
backport-to-v4-next-stagingfrom
claudebox/backport-22050-pxe-warn

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

Backport of #22050 to v4-next.

Cherry-pick applied cleanly. The noir-projects change from the original PR was already present on v4-next, so only the PXE block_synchronizer changes were needed.

Changes

  • Adds warning logs in PXE block synchronizer when getBlockHeader returns null for chain-proven and chain-finalized events, instead of silently skipping the anchor update
  • Adds a comment clarifying the local promise capture pattern in sync()

ClaudeBox log: https://claudebox.work/s/017ec0a44d097055?run=1

…nts (#22050)

Improved logging based on audit finding 114 in #21500 + random
improvements that I noticed could be done

## Summary
- Adds warning logs in PXE block synchronizer when `getBlockHeader`
returns null for `chain-proven` and `chain-finalized` events, instead of
silently skipping the anchor update.
- Aligns with the `chain-pruned` handler which already throws on missing
headers.

Closes #21500 .
@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 15, 2026
@benesjan benesjan marked this pull request as ready for review April 15, 2026 05:48
@benesjan benesjan enabled auto-merge April 15, 2026 05:49
@benesjan benesjan merged commit 0ea98ea into backport-to-v4-next-staging Apr 15, 2026
20 of 26 checks passed
@benesjan benesjan deleted the claudebox/backport-22050-pxe-warn branch April 15, 2026 06:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants