Skip to content

fix: unify jussi_num to last_irreversible_block_num on /health and /#261

Merged
ety001 merged 2 commits into
nextfrom
pr/07-unify-jussi-num-to-lib
Jun 1, 2026
Merged

fix: unify jussi_num to last_irreversible_block_num on /health and /#261
ety001 merged 2 commits into
nextfrom
pr/07-unify-jussi-num-to-lib

Conversation

@ety001

@ety001 ety001 commented Jun 1, 2026

Copy link
Copy Markdown
Member

Both /health and / (homepage) endpoints now return last_irreversible_block_num instead of head_block_number, matching jussi legacy behavior. The value is sourced from BlockNumberTracker which is passively updated by UpdateBlockNumberMiddleware from get_dynamic_global_properties responses.

Removed GlobalParams (global_params.go) which actively polled steemd for head_block_number — no longer needed and was returning the wrong field.

@ety001 ety001 force-pushed the pr/07-unify-jussi-num-to-lib branch from fe28801 to ccd0dcc Compare June 1, 2026 17:44
ety001 added 2 commits June 2, 2026 01:44
Both /health and / (homepage) endpoints now return last_irreversible_block_num
instead of head_block_number, matching jussi legacy behavior. The value is
sourced from BlockNumberTracker which is passively updated by
UpdateBlockNumberMiddleware from get_dynamic_global_properties responses.

Removed GlobalParams (global_params.go) which actively polled steemd for
head_block_number - no longer needed and was returning the wrong field.
@ety001 ety001 merged commit 566cbb9 into next Jun 1, 2026
2 checks passed
@ety001 ety001 deleted the pr/07-unify-jussi-num-to-lib branch June 1, 2026 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant