Fix Synchronizer not setting slot.ForgerCommitment to true until fully synced and after reorgs
Oleksandr Brezhniev oleksandr.brezhniev@gmail.com
Wed, 17 Mar 2021 13:21:27 +0200
1 files changed,
3 insertions(+),
5 deletions(-)
jump to
M
synchronizer/synchronizer.go
→
synchronizer/synchronizer.go
@@ -354,15 +354,13 @@ slot.ForgerCommitment = false
} slot.SlotNum = slotNum slot.StartBlock, slot.EndBlock = s.consts.Auction.SlotBlocks(slot.SlotNum) + if hasBatch && s.consts.Auction.RelativeBlock(firstBatchBlockNum) < int64(s.vars.Auction.SlotDeadline) { + slot.ForgerCommitment = true + } // If Synced, update the current coordinator if s.stats.Synced() && blockNum >= s.consts.Auction.GenesisBlockNum { if err := s.setSlotCoordinator(slot); err != nil { return tracerr.Wrap(err) - } - if hasBatch && - s.consts.Auction.RelativeBlock(firstBatchBlockNum) < - int64(s.vars.Auction.SlotDeadline) { - slot.ForgerCommitment = true } // TODO: Remove this SANITY CHECK once this code is tested enough