Call ensureConnection in PeerID to ensure we have fetched the certs (… (#189)

cmmarslender · web-flow · commit b7bffe02b1f3 · 2025-01-14T08:56:56.000-07:00
diff --git a/pkg/peerprotocol/connection.go b/pkg/peerprotocol/connection.go
@@ -167,8 +167,13 @@ func (c *Connection) Close() {
 }
 
 // PeerID returns the Peer ID for the remote peer
-func (c *Connection) PeerID() types.Bytes32 {
-	return c.peerID
+func (c *Connection) PeerID() (types.Bytes32, error) {
+	err := c.ensureConnection()
+	if err != nil {
+		return types.Bytes32{}, err
+	}
+
+	return c.peerID, nil
 }
 
 // Handshake performs the RPC handshake. This should be called before any other method

Original file line number	Diff line number	Diff line change
`@@ -167,8 +167,13 @@ func (c *Connection) Close() {`
`167`	`167`	`}`
`168`	`168`
`169`	`169`	`// PeerID returns the Peer ID for the remote peer`
`170`		`-func (c *Connection) PeerID() types.Bytes32 {`
`171`		`- return c.peerID`
	`170`	`+func (c *Connection) PeerID() (types.Bytes32, error) {`
	`171`	`+ err := c.ensureConnection()`
	`172`	`+ if err != nil {`
	`173`	`+ return types.Bytes32{}, err`
	`174`	`+ }`
	`175`	`+`
	`176`	`+ return c.peerID, nil`
`172`	`177`	`}`
`173`	`178`
`174`	`179`	`// Handshake performs the RPC handshake. This should be called before any other method`