[PATCH]: IPv6 SMTP proxy with XCLIENT

[Maxim Dounin]

Hello!

On Mon, Aug 22, 2011 at 02:45:19PM +0200, Brane F. Gračnar wrote:

> Hello!
> 
> There is a issue with XCLIENT support when nginx listener is IPv6 enabled. 
> Nginx sends the following command to upstream SMTP server:
> 
> XCLIENT ADDR=2001:15c0:65ff:xxxx::yyyy NAME=[UNAVAILABLE]
> 
> According to postfix docs (http://www.postfix.org/XCLIENT_README.html) IPv6 
> client addresses should be prefixed with "IPV6:" string, so nginx should send 
> the following xclient command to upstream:
> 
> XCLIENT ADDR=IPV6:2001:15c0:65ff:xxxx::yyyy NAME=[UNAVAILABLE]
> 
> 
> Attached patch fixes this issue. It would be great if it would be included in 
> upstream after careful review.

Thank you, the problem is definitely valid, though I don't like 
the patch.  It should check address family instead of searching 
for ":" in address text representation.  I'll take a look as time 
permits.

Maxim Dounin