Skip to content

Commit b090ad2

Browse files
committed
refactor(client): extract address parsing into separate function
1 parent 5a1a2d9 commit b090ad2

File tree

1 file changed

+13
-9
lines changed

1 file changed

+13
-9
lines changed

client.go

Lines changed: 13 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -783,6 +783,18 @@ func savePeerCache(peers []cachedPeer, cacheFile string, logger logger) {
783783
}
784784
}
785785

786+
func parseMultiaddrs(addrs []string) []multiaddr.Multiaddr {
787+
maddrs := make([]multiaddr.Multiaddr, 0, len(addrs))
788+
for _, addrStr := range addrs {
789+
maddr, err := multiaddr.NewMultiaddr(addrStr)
790+
if err != nil {
791+
continue
792+
}
793+
maddrs = append(maddrs, maddr)
794+
}
795+
return maddrs
796+
}
797+
786798
func connectToCachedPeers(ctx context.Context, h host.Host, cachedPeers []cachedPeer, logger logger) {
787799
for _, cp := range cachedPeers {
788800
peerID, err := peer.Decode(cp.ID)
@@ -795,15 +807,7 @@ func connectToCachedPeers(ctx context.Context, h host.Host, cachedPeers []cached
795807
continue
796808
}
797809

798-
var maddrs []multiaddr.Multiaddr
799-
for _, addrStr := range cp.Addrs {
800-
maddr, err := multiaddr.NewMultiaddr(addrStr)
801-
if err != nil {
802-
continue
803-
}
804-
maddrs = append(maddrs, maddr)
805-
}
806-
810+
maddrs := parseMultiaddrs(cp.Addrs)
807811
if len(maddrs) == 0 {
808812
continue
809813
}

0 commit comments

Comments
 (0)