more cleanups

osmosis-labs · Jun 24, 2024 · 5062431 · 5062431
1 parent c6d0307
commit 5062431
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 17 deletions.
diff --git a/p2p/mock/peer.go b/p2p/mock/peer.go
@@ -72,4 +72,3 @@ func (mp *Peer) RemoteAddr() net.Addr        { return &net.TCPAddr{IP: mp.ip, Po
 func (mp *Peer) CloseConn() error            { return nil }
 func (mp *Peer) SetRemovalFailed()           {}
 func (mp *Peer) GetRemovalFailed() bool      { return false }
-func (mp *Peer) GetRegion() string           { return "" }
diff --git a/p2p/mocks/peer.go b/p2p/mocks/peer.go
diff --git a/p2p/pex/addrbook.go b/p2p/pex/addrbook.go
@@ -1071,8 +1071,10 @@ func getRegionFromIP(ip string) (string, error) {
 	}
 
 	var ipInfo ipInfo
-	json.Unmarshal(body, &ipInfo)
-	// fmt.Println("ipInfoPeer", ipInfo)
+	err = json.Unmarshal(body, &ipInfo)
+	if err != nil {
+		return "", err
+	}
 
 	if ipInfo.Status != "success" {
 		return "", fmt.Errorf("failed to get country from IP %s", ip)

diff --git a/p2p/pex/pex_reactor.go b/p2p/pex/pex_reactor.go
@@ -580,6 +580,7 @@ func (r *Reactor) ensurePeersCommon(regionAware bool) {
 	}
 
 	if r.book.NeedMoreAddrs() {
+
 		// 1) Pick a random peer and ask for more.
 		peers := r.Switch.Peers().List()
 		peersCount := len(peers)