GeoIPv6 patch

Gregor Kališnik gregor at unimatrix-one.org
Tue Jun 14 09:52:10 MSD 2011


Hi.

On Tuesday 14 of June 2011 05:38:59 you wrote:
> Hello!
> 
> On Sat, Jun 11, 2011 at 08:19:52PM +0200, Gregor Kališnik wrote:
> > Hi.

> You can't assume you have sockaddr_in6 unless compiled with ipv6.

If IPv6 is not compiled, then GeoIPv6 also won't be compiled.

> This should go to auto/lib/geoip/conf.  And probably should be
> made shorter.
> 
> Naming this NGX_HAVE_GEOIP_IPV6 would be better IMHO.

I'm checking for all used variables. Is it sufficient to look just for one?


> 
> How this may happen and what's the expected result?
> 

The whole ngx_http_geoip_addr_v6 is now absent from no-ipv6 version.


Best regards,
Gregor Kališnik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: nginx-geoipv6.patch
Type: text/x-patch
Size: 8255 bytes
Desc: not available
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110614/87ab8bcf/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part.
URL: <http://nginx.org/pipermail/nginx-devel/attachments/20110614/87ab8bcf/attachment.pgp>


More information about the nginx-devel mailing list