[Patch][GeoIP][1.0.0] GeoIP Organization / ISP support

Igor Sysoev igor at sysoev.ru
Mon Apr 18 22:20:57 MSD 2011


On Mon, Apr 18, 2011 at 05:56:09PM +0300, Arnaud GRANAL wrote:
> Igor Sysoev wrote:
> >  There is already patch to support GeoIP ISP base:
> >  http://kornel.neolabs.kz/files/nginx-0.9.7-geoip-isp.patch.bz2
> 
> >  What is the difference between ISP base and Organization base ?
> 
> Almost none, organization is more precise but API calls are the same.
> As far I can see the code is very similar so there is no reason to use my patch.

To get GeoIP_1st patch calls GeoIP_name_by_ipnum(),
while your patch calls GeoIP_org_by_ipnum() to get organization.
Do these capabilities exist in both bases ?

> I didn't see that this patch exist so I spent a little bit time coding myself,
> to save some time to other people could you consider putting the 0.9.7 patch into main tree ?

This patch was in Russian mailing list so you did not see it.
I think it's better to combine ISP and organization capability.


-- 
Igor Sysoev



More information about the nginx-devel mailing list