Maniphest T119801

Update GeoIP cookie to include netspeed info
Closed, DeclinedPublic
Actions

Assigned To

Authored By

	ori
	Nov 28 2015, 7:41 PM

Description

The GeoIP C API that we use in VCL to geolocate the client IP address supports using the MaxMind Netspeed database to determine the client connection speed.

Example code:
https://github.com/maxmind/geoip-api-c/blob/master/test/test-geoip-netspeed.c

We already have a subscription for the database (@kevinator or @Nuria should know the details), so it is presumably a matter of deploying the database files to the cache boxes and updating the VCL code in operations/puppet.git:templates/varnish/geoip.inc.vcl.erb to utilize it.

See T119797: Serve low-res images by default to users on slow or metered mobile connections for use-case.

Related Objects
Search...

Status	Subtype	Assigned	Task
Open	Release	None	T84936 Release VisualEditor-MediaWiki as "1.0"
Open		None	T50429 [Epic] Support editing parts of a page in VisualEditor-MediaWiki
Open		None	T54365 Explore performance gains from progressive (JIT?) de-alienation in VisualEditor
Open		None	T396539 Introduce a way to edit a span of text from within read mode
Open		None	T385558 Improve copy and paste from read mode to VE
Open		None	T174303 Copy-pasting linked ISBN numbers from view mode HTML into VisualEditor inserts wikitext links to Special:BookSources (it should turn them into magic links?)
Open	Feature	None	T54091 The read HTML should have hinting to allow full DOM copying (as opposed to just rich copying) from read mode into VE surfaces
Open		None	T55784 [EPIC] Use Parsoid HTML for all page views
Resolved		dr0ptp4kt	T114542 Next Generation Content Loading and Routing, in Practice
Duplicate		None	T36023 "lazy" image loading
Duplicate		• Jhernandez	T104432 [EPIC]: Improve mobile site performance
Duplicate		dr0ptp4kt	T120341 [GOAL] Make Wikipedia more accessible to all connections with new fast API-driven web experience in mobile web beta
Declined		None	T125920 [EPIC] Future exciting reading web performance endeavours
Resolved		Jdlrobson	T113066 [GOAL] Make Wikipedia more accessible to 2G connections
Declined		None	T120875 Experiment with serving lower resolution images as the default for all page views
Invalid		None	T117001 Increase mobile performance for 2G connections
Declined		None	T114791 Explore lazy-loading WebP thumbnails to supporting browsers (eg Android Chrome)
Resolved		ori	T119797 Serve low-res images by default to users on slow or metered mobile connections
Declined		None	T119798 Vary mobile HTML by connection speed
Declined		ori	T119801 Update GeoIP cookie to include netspeed info

Event Timeline

ori created this task.Nov 28 2015, 7:41 PM

ori raised the priority of this task from to High.

ori updated the task description. (Show Details)

ori added projects: Varnish, Web-Team-Backlog-Archived, Performance-Team.

ori added subscribers: BBlack, • Niedzielski, Volker_E and 12 others.

The text Varnishes are already Puppetized to receive the latest version of the netspeed database into /usr/share/GeoIP, yay.

According to MaxMind, the NetSpeed database classifies connection as either dial-up, cable/DSL, corporate or cellular. If all cellular connections get mapped to "cellular", this is of limited value.

Yeah, and also note "[t]he connection type is about 95% accurate in the US. Outside the US, accuracy ranges from 50% to 80%, depending on the country. The data is generally more accurate for countries with more Internet users." which would possibly exclude places that typically have lower connection speeds.

In any case, see also T99226 and https://gerrit.wikimedia.org/r/#/c/253619/ — we're going to eventually drop GeoIP 1 in favor of libmaxminddb/GeoIP2. The fundraiser is a bad time to be messing with the GeoIP cookie though. There is a GeoIP2 equivalent to the NetSpeed database called "connection type".

Neither the legacy nor the new-and-improved database from MaxMind breaks down mobile connections into different types, which is what we really need, so this is a non-starter. I propose we use geolocation for T119798 instead.

ori closed this task as Declined.Nov 30 2015, 7:18 AM

ori claimed this task.

Aklapper added a project: Traffic.May 4 2016, 9:13 AM

Restricted Application added a project: SRE. · View Herald TranscriptMay 4 2016, 9:13 AM

Nemo_bis subscribed.Aug 18 2016, 5:51 AM

Krinkle mentioned this in T176460: Test out the new Netinfo API coming in Chrome 62.Sep 27 2017, 7:45 PM