github/lodestar
1
1
Fork 0
mirror of https://github.com/ChainSafe/lodestar.git synced 2026-01-10 08:08:16 -05:00
Code Issues Packages Projects Releases Wiki Activity

fix: use correct container and bump seq_number when updating syncnets (#8451)

Browse Source
This commit is contained in:
Nico Flaig
2025-09-23 13:14:41 +01:00
committed by GitHub
parent 4efea5815e
commit b8be78f7bb
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/beacon-node/src/network/metadata.ts
View File

@@ -64,7 +64,7 @@ export class MetadataController {
if (config.getForkSeq(computeStartSlotAtEpoch(currentEpoch)) >= ForkSeq.altair) {
// Only persist syncnets if altair fork is already activated. If currentFork is altair but head is phase0
// adding syncnets to the ENR is not a problem, we will just have a useless field for a few hours.
this.onSetValue(ENRKey.syncnets, ssz.phase0.AttestationSubnets.serialize(this._metadata.syncnets));
this.onSetValue(ENRKey.syncnets, ssz.altair.SyncSubnets.serialize(this._metadata.syncnets));
}
// Set CGC regardless of fork. It may be useful to clients before Fulu, and will be ignored otherwise.
@@ -81,6 +81,7 @@ export class MetadataController {
set syncnets(syncnets: BitArray) {
this.onSetValue(ENRKey.syncnets, ssz.altair.SyncSubnets.serialize(syncnets));
this._metadata.seqNumber++;
this._metadata.syncnets = syncnets;
}