Use
net.JoinHostPort instead of fmt.Sprintf(...) GO-S1027Use net.JoinHostPort instead of: fmt.Sprintf("%s:%s", localBindAddr, localPort)
99
100 if _, err := strconv.Atoi(localPort); err == nil {
101 // just numbers
102 addr, err := net.ResolveTCPAddr("tcp", fmt.Sprintf("%s:%s", localBindAddr, localPort))103 if err != nil {
104 return nil, err
105 }
Description
Instead of manually joining hostname and port combinations, it is better to use net.JoinHostPort.
Bad practice
fmt.Sprintf("%s:%s", host, port)
fmt.Sprintf("%s:%d", host, port)
fmt.Sprintf("%s:%s", "", host)
fmt.Sprintf("%s:%d", "", host)
Recommended
net.JoinHostPort(host, port)
net.JoinHostPort(host, port)
net.JoinHostPort(host, "")
net.JoinHostPort(host, "")