Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include consensus_hash in /new_burn_block and /new_block #5590

Open
hstove opened this issue Dec 18, 2024 · 0 comments
Open

Include consensus_hash in /new_burn_block and /new_block #5590

hstove opened this issue Dec 18, 2024 · 0 comments

Comments

@hstove
Copy link
Contributor

hstove commented Dec 18, 2024

I think it would be helpful to add consensus_hash to both the /new_burn_block and /new_block event dispatcher payloads.

The reason for this is somewhat selfish - when building slot watch, I try and correlate block proposals to burn blocks. But block proposals only have consensus_hash (and burn height), and I'm trying to only use event observer data, so I don't have a great way of mapping a proposal to a burn block.

If we add consensus_hash to these payloads, it'll make it really easy to correlate proposals to burn blocks without needing to rely on RPC requests.

This might also be helpful for the signer - it could "eagerly" update the burn tip's consensus hash without needing to make an RPC request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Status: 🆕 New
Development

No branches or pull requests

1 participant