[tor-commits] [tor/master] Fall back to registered country if necessary.

[nickm@xxxxxxxxxxxxxx]

commit 1d2179bc900f1646a5491b65294e78b175e70056
Author: Karsten Loesing <karsten.loesing@xxxxxxx>
Date:   Tue Feb 25 13:20:04 2014 +0100

    Fall back to registered country if necessary.
    
    When extracting geoip and geoip6 files from MaxMind's GeoLite2 Country
    database, we only look at country->iso_code which is the two-character ISO
    3166-1 country code of the country where MaxMind believes the end user is
    located.
    
    But if MaxMind thinks a range belongs to anonymous proxies, they don't put
    anything there.  Hence, we omit those ranges and resolve them all to '??'.
    That's not what we want.
    
    What we should do is first try country->iso_code, and if there's no such
    key, try registered_country->iso_code which is the country in which the
    ISP has registered the IP address.
    
    In short: let's fill all A1 entries with what ARIN et. al think.
---
 src/config/mmdb-convert.py |   13 +++++++++++++
 1 file changed, 13 insertions(+)

diff --git a/src/config/mmdb-convert.py b/src/config/mmdb-convert.py
index 21d170a..4245738 100644
--- a/src/config/mmdb-convert.py
+++ b/src/config/mmdb-convert.py
@@ -339,11 +339,24 @@ def parse_mm_file(s):
 def format_datum(datum):
     """Given a Datum at a leaf of the tree, return the string that we should
        write as its value.
+
+       We first try country->iso_code which is the two-character ISO 3166-1
+       country code of the country where MaxMind believes the end user is
+       located.  If there's no such key, we try registered_country->iso_code
+       which is the country in which the ISP has registered the IP address.
+       Without falling back to registered_country, we'd leave out all ranges
+       that MaxMind thinks belong to anonymous proxies, because those ranges
+       don't contain country but only registered_country.  In short: let's
+       fill all A1 entries with what ARIN et. al think.
     """
     try:
         return bytesToStr(datum.map['country'].map['iso_code'].data)
     except KeyError:
         pass
+    try:
+        return bytesToStr(datum.map['registered_country'].map['iso_code'].data)
+    except KeyError:
+        pass
     return None
 
 IPV4_PREFIX = "0"*96



_______________________________________________
tor-commits mailing list
tor-commits@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-commits