Remove incomplete and dated 'wgUseETag' experiment
authorTimo Tijhof <krinklemail@gmail.com>
Mon, 27 Jun 2016 11:21:14 +0000 (13:21 +0200)
committerKrinkle <krinklemail@gmail.com>
Mon, 27 Jun 2016 12:43:54 +0000 (12:43 +0000)
commit46df493ac069b48dff7673d9b2cf24aae891a0fc
treefca344d6b8dc95dcdf22b7ad2a3c15d9a3d6cdf4
parent6af8dcf5763a67315d3b6e52fdea27fe7df952b5
Remove incomplete and dated 'wgUseETag' experiment

Follows-up:
8ff36ebd (r9793) Set ETag header on article views.
8167b89e (r16205) Add $wgUseETag, false by default (broke Squid; T9098).

ETag is a useful alternative for If-Modified-Since (see T33639, T49529 about
use cases and bugs it may fix). However the current incomplete implementation
isn't useful.

It's a feature flag that has been disabled since 2006. Enabling it stores
ParserCache::getETag in OutputPage and sends it out as a response header.
It's ignored on incoming requests. We don't check If-None-Match for
'304 Not Modified' anywhere.

ResourceLoader has a working implementation that could be leveraged.

To be re-evaluated in the future. ParserCache::getETag() could use
improvement as well.

Change-Id: I37fd9d0ce1ce9bffb79243c6ae340403fdfd769f
includes/DefaultSettings.php
includes/OutputPage.php
includes/page/Article.php