Hi Matthias,

> 1. Remove ipv6only and make IPV6_V6ONLY default as specified in
> RFC 4038.

"ipv6only" should be deprecated (ignored) and not removed.

> 2. Use getaddrinfo() instead of gethostbyname().

Agreed, but it should also fall-back to gethostbyname2() and finally to
gethostbyname().

(Are there any systems that have gethostbyname2() but don't have
getaddrinfo()?)

> 3. Listen on multiple IPv4 or IPv6 addresses if a hostname was
> given in a listen directive.

Agreed.

> 4. Add the options “ipv4” and “ipv6” to the listen directive
> to set ai_family to AF_INET or AF_INET6 in struct addrinfo. This
> could be useful if you want to serve a different site depending
> on the address family the request came from or to simple disable
> IPv6 or IPv4 even if the hostname has an associated address.

Agreed.

> getaddrinfo() can resolve services and numeric addresses itself, so I
> don't see need for the square bracket notation as in RFC 3986, a simple
> space would be enough and would make parsing much easier:
>
> listen 2001:db8::1 www;

Yeah, I don't see the need for square brackets.

> What do you think about this?

Do you have patch for this? :)

Best regards,
Piotr Sikora < piotr.sikora@frickle.com >



_______________________________________________
nginx-devel mailing list
nginx-devel@nginx.org
http://nginx.org/mailman/listinfo/nginx-devel