[Patch][GeoIP][1.0.0] GeoIP Organization / ISP support
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.
More information about the nginx-devel