Reduce redundant logic in validateAddressFormat

This commit is contained in:
eyedeekay
2025-05-26 22:36:53 -04:00
parent fdb8745ee0
commit ca5c7feb9f

View File

@ -62,13 +62,11 @@ func sanitizeAddress(addr string) string {
func validateAddressFormat(addr string) error { func validateAddressFormat(addr string) error {
host, _, err := net.SplitHostPort(addr) host, _, err := net.SplitHostPort(addr)
if err != nil { if err == nil {
// If SplitHostPort fails, it means addr is not in host:port format // Successfully split host:port, use just the host part
host = addr
}
if host != "" {
addr = host addr = host
} }
if len(addr) > MaxAddressLength || len(addr) < MinAddressLength { if len(addr) > MaxAddressLength || len(addr) < MinAddressLength {
return fmt.Errorf("invalid address length: got %d, want between %d and %d", return fmt.Errorf("invalid address length: got %d, want between %d and %d",
len(addr), MinAddressLength, MaxAddressLength) len(addr), MinAddressLength, MaxAddressLength)