Skip to content

Commit 97aaecc

Browse files
committed
feat(parachain): Auto-detect V2 peers from AdvertiseCollationV2 (#4711)
- Upgrade peer to ProtocolV2 when receiving V2 advertisement - Remove redundant V1 default in PeerConnected handler - Complete automatic version detection for collation requests
1 parent a49d431 commit 97aaecc

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

dot/parachain/collator-protocol/validator-side/message.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -449,6 +449,9 @@ func (cpvs *CollatorProtocolValidatorSide) processCollatorProtocolMessage(sender
449449
return fmt.Errorf("handling v2 advertisement: %w", err)
450450
}
451451

452+
logger.Debugf("Peer %s sent V2 advertisement, upgrading to ProtocolV2", sender)
453+
cpvs.setPeerProtocolVersion(sender, ProtocolV2)
454+
452455
case 4: // CollationSeconded
453456
logger.Errorf("unexpected collation seconded message from peer %s, decreasing its reputation", sender)
454457
cpvs.SubSystemToOverseer <- networkbridgemessages.ReportPeer{

dot/parachain/collator-protocol/validator-side/validator_side.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -858,8 +858,6 @@ func (cpvs *CollatorProtocolValidatorSide) handleNetworkBridgeEvents(msg any) er
858858
Instant: time.Now(),
859859
},
860860
}
861-
// Default to V1, will upgrade if we detect V2 capabilities later
862-
cpvs.setPeerProtocolVersion(msg.PeerID, ProtocolV1)
863861
}
864862
case networkbridgeevents.PeerDisconnected:
865863
delete(cpvs.peerData, msg.PeerID)

0 commit comments

Comments
 (0)