Page MenuHomePhabricator

Strip query string in varnish upload
Closed, ResolvedPublic

Description

There have been cases where swift machines would get the 1G NIC close to saturation due to repeated hits for the same file. The ultimate reason for the repeated hits is files being requested with cache-busting query strings, usually with value being the unix timestamp in (milli)seconds (e.g. T150551).
For upload varnish we can strip the query string altogether since it isn't used by mw / thumbor. Note this might change in the future depending on T66214: Define an official thumb API outcome but we can do this now.

Related Objects

Event Timeline

fgiunchedi renamed this task from String query string in varnish upload to Strip query string in varnish upload.Dec 15 2016, 5:48 PM

Change 327553 had a related patch set uploaded (by BBlack):
cache_upload: strip all query params

https://gerrit.wikimedia.org/r/327553

Change 327553 merged by BBlack:
cache_upload: strip all query params

https://gerrit.wikimedia.org/r/327553

BBlack claimed this task.