ServerAddress should use std::net types to pass a TCP socket.

[JonathanWoollett-Light]

ServerAddress should make use of existing standard library types to pass the TCP socket e.g. SocketAddr. Passing it as a string (e.g. ServerAddress::Tcp { host "my_socket", port: Some(4040) }) is error prone.

[abr-egn]

Just to make sure I understand, your request is for an IP address specific variant of ServerAddress, e.g.

pub enum ServerAddress {
  Tcp { ... },
  Unix { ... },
  SocketAddr(std::net::SocketAddr),
}

?

[JonathanWoollett-Light]

That looks good, really any approach that passes the socket as std::net types rather than a String and a Option<u16>. Currently I'm doing Tcp { host: addr.to_string(), .. } which is pretty ugly.