Connection Information may contain a hostname

[JonathanWilbur]

From IETF RFC 8866:

connection-field =    %s"c" "=" nettype SP addrtype SP connection-address CRLF

[...]

; sub-rules of 'c='
connection-address =  multicast-address / unicast-address

[...]

; generic sub-rules: addressing
unicast-address =     IP4-address / IP6-address / FQDN / extn-addr

multicast-address =   IP4-multicast / IP6-multicast / FQDN / extn-addr

[...]

extn-addr =  non-ws-string

But the type for ConnectionAddress.base is IpAddr.

I'd be happy to create a PR to fix this if you'll publish it when approved / merged.

[JonathanWilbur]

I think it should have a type like

pub enum ConnectionAddressBase {
    Ip(IpAddr),
    Fqdn(String),
    Extn(String)
}

I feel like Fqdn above could use stronger typing, which I know you value in your crates, but I'm not sure what it could be other than Vec<String> to represent the DNS labels, which is more abstruse than it is worth, IMO.

[JonathanWilbur]

Oh, I also just discovered this is a problem with the Origin.unicast_address type.