-
Notifications
You must be signed in to change notification settings - Fork 2
fix: update chainhook predicate start_block upon re-registration #308
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
Conversation
Vercel deployment URL: https://token-metadata-pgqyxgdca-hirosystems.vercel.app 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, added 2 non-blocking comments for possible changes if warranted.
## [1.2.1-beta.1](v1.2.0...v1.2.1-beta.1) (2025-06-04) ### Bug Fixes * update chainhook predicate start_block upon re-registration ([#308](#308)) ([929f08e](929f08e))
🎉 This PR is included in version 1.2.1-beta.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
🎉 This PR is included in version 1.3.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
If a chainhook predicate goes obsolete, update its starting block height on re-registration.
There is a bug on chainhook which sometimes marks active predicates as inactive, forcing the Metadata API to re-register so it receives blocks again. However, since we weren't updating the re-registration starting block height, we ended up in an infinite loop that made us receive blocks from the last block height seen on boot. This caused the API to fall significantly behind chain tip.