Module: sip-router Branch: master Commit: b280f89b3ac4f97c3cad1487961509bae893f656 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b280f89b...
Author: Henning Westerholt henning.westerholt@1und1.de Committer: Henning Westerholt henning.westerholt@1und1.de Date: Mon May 16 15:48:53 2011 +0200
geoip: extend docs after discussion with Francesco Castellano
---
modules/geoip/README | 8 ++++++++ modules/geoip/doc/geoip_admin.xml | 11 ++++++++++- 2 files changed, 18 insertions(+), 1 deletions(-)
diff --git a/modules/geoip/README b/modules/geoip/README index 016d0d8..595b5df 100644 --- a/modules/geoip/README +++ b/modules/geoip/README @@ -82,6 +82,14 @@ Chapter 1. Admin Guide and its derivatives as libgeoip, and probably by other distributions as well.
+ Debian Linux squeeze includes already a database as dependency, but as + this contain the wrong data, it will not work correctly with the + module. More acurate, the module expect the GeoIP City Edition, and + will not work with the GeoIP Country Edition. In newer Debian Linux + releases the package geoip-database-contrib should contain the + necessary database. You can download the Lite edition of the DB from + here. + This module exports a new class of pseudo-variables - $gip(pvc=>key) - to enable access to the results of a query to the database.
diff --git a/modules/geoip/doc/geoip_admin.xml b/modules/geoip/doc/geoip_admin.xml index b8c9f17..f1df9ca 100644 --- a/modules/geoip/doc/geoip_admin.xml +++ b/modules/geoip/doc/geoip_admin.xml @@ -28,7 +28,7 @@ </para> <para> This database itself can be obtained on a free or commercial basis - <ulink url="http://www.maxmind.com/app/ip-location">here</ulink>. The + <ulink url="http://www.maxmind.com/app/ip-location">here</ulink>. The library that interfaces with the Max Mind API, as well as scripts to automate downloading of the on-disk version of the open-source database is also packaged by the Debian Linux distribution and @@ -36,6 +36,15 @@ other distributions as well. </para> <para> + Debian Linux squeeze includes already a database as dependency, but as + this contain the wrong data, it will not work correctly with the module. + More acurate, the module expect the <emphasis>GeoIP City Edition</emphasis>, + and will not work with the <emphasis>GeoIP Country Edition</emphasis>. In + newer Debian Linux releases the package <emphasis>geoip-database-contrib</emphasis> + should contain the necessary database. You can download the Lite edition + of the DB from <ulink url="http://www.maxmind.com/app/geolitecity">here</ulink>. + </para> + <para> This module exports a new class of pseudo-variables - $gip(pvc=>key) - to enable access to the results of a query to the database.
On 5/16/11 3:49 PM, Henning Westerholt wrote:
Module: sip-router Branch: master Commit: b280f89b3ac4f97c3cad1487961509bae893f656 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=b280f89b...
Author: Henning Westerholthenning.westerholt@1und1.de Committer: Henning Westerholthenning.westerholt@1und1.de Date: Mon May 16 15:48:53 2011 +0200
geoip: extend docs after discussion with Francesco Castellano
modules/geoip/README | 8 ++++++++ modules/geoip/doc/geoip_admin.xml | 11 ++++++++++- 2 files changed, 18 insertions(+), 1 deletions(-)
diff --git a/modules/geoip/README b/modules/geoip/README index 016d0d8..595b5df 100644 --- a/modules/geoip/README +++ b/modules/geoip/README @@ -82,6 +82,14 @@ Chapter 1. Admin Guide and its derivatives as libgeoip, and probably by other distributions as well.
- Debian Linux squeeze includes already a database as dependency, but as
- this contain the wrong data, it will not work correctly with the
- module. More acurate, the module expect the GeoIP City Edition, and
- will not work with the GeoIP Country Edition. In newer Debian Linux
- releases the package geoip-database-contrib should contain the
- necessary database. You can download the Lite edition of the DB from
- here.
probably full link value should be given instead of word 'here' so that the text format for README has it as well.
Cheers, Daniel
Hello,
On 5/16/11 3:55 PM, Henning Westerholt wrote:
On Monday 16 May 2011, Daniel-Constantin Mierla wrote:
probably full link value should be given instead of word 'here' so that the text format for README has it as well.
Hi Daniel,
i think the URL is only included in the website, as a proper link.
for that reason I think it would be more useful to replace the word 'here' with the link:
<ulink url="http://www.maxmind.com/app/ip-location">http://www.maxmind.com/app/ip-location </ulink>
In this way it will appear in the text README.
Cheers, Daniel
On Monday 16 May 2011, Daniel-Constantin Mierla wrote:
i think the URL is only included in the website, as a proper link.
for that reason I think it would be more useful to replace the word 'here' with the link:
<ulink url="http://www.maxmind.com/app/ip-location">http://www.maxmind.com/app/ip -location </ulink>
Hi Daniel,
good idea - fixed.
Cheers,
Henning
-
Daniel-Constantin Mierla -
Henning Westerholt