Opened 3 years ago

Closed 3 years ago

#2268 closed defect (fixed)

Replace GeoIP source/format

Reported by: zzz Owned by: zzz
Priority: minor Milestone: 0.9.38
Component: router/transport Version: 0.9.34
Keywords: Cc:
Parent Tickets: Sensitive: no


We use MaxMind? "GeoLite? Legacy"
IPv4 not updated after April 1 2018.
IPv6 last update was June 4, but will go away too.

It appears we can switch to their version 2 format, need to verify license and see if the maxmind code we already have supports it. Benefit is version 2 is much faster. Or we could offline convert it back to legacy format, but it's much slower to read.

Also see Tor reactions:
Seems like they already switched to version 2


Change History (4)

comment:1 Changed 3 years ago by zzz

Status: newaccepted

Version 2 code is same license, Apache 2.
Base source code (geoip2 and db in separate trees) is about 160KB source compared to 60KB for version 1.
But requires about 1.7 MB of jars in dependencies for "jackson" JSON lib including annotations.
The database is not in JSON format, but the code uses json as an internal representation. The database Reader class returns a jackson JSON object.
Can probably convert to use the net.minidev.json we checked in for DoH, or another internal format. To be investigated.

Was thinking maybe this would be easy enough to pull into 36, but now seems unlikely.

comment:3 Changed 3 years ago by zzz

Milestone: 0.9.370.9.38

comment:4 Changed 3 years ago by zzz

Resolution: fixed
Status: acceptedclosed
Note: See TracTickets for help on using tickets.