HomePhabricator

varnish: switch from libGeoIP to libmaxminddb
8a7940867cd8Unpublished

Unpublished Commit · Learn More

Not On Permanent Ref: This commit is not an ancestor of any permanent ref.
This commit no longer exists in the repository. It may have been part of a branch which was deleted.This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.

Description

varnish: switch from libGeoIP to libmaxminddb

This switches our Varnish from the old libgeoip to libmaxminddb,
which uses the new GeoIP2 databases and adds IPv6 support to the
GeoIP Set-Cookie and our geoiplookup JSON.

The region identifier is different than the old format, which may
cause some breakage; the old format was a numeric identifier in
the database, while the new format is a GeoIP ISO code.

Also, the final element of the cookie no longer differentiates
IPv4 vs IPv6 (which was primarily for related fallback), so the
final field is always 'v4' now.

One notable deficiency here is that updated MaxMind databases are
only reloaded when VCL is reloaded, but I think we can save that
problem for later varnish4/vmod work.

Bug: T99226
Change-Id: I7685edd63a0dcddcd32d950f563e8f0e51dfd281

Details

Provenance
faidonAuthored on Nov 17 2015, 3:42 PM
BBlackCommitted on Aug 19 2016, 1:32 PM
ChangeId
I7685edd63a0dcddcd32d950f563e8f0e51dfd281

Commit No Longer Exists

This commit no longer exists in the repository.