From 51b959dcdc3f1275c518822a1c183711f517cebd Mon Sep 17 00:00:00 2001 From: Roan Kattouw Date: Mon, 14 Mar 2011 15:16:43 +0000 Subject: [PATCH] (bug 27564) Timestamp of site module embedded in Squid-cached page HTML --- includes/OutputPage.php | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/includes/OutputPage.php b/includes/OutputPage.php index 576be09e5c..b4059fe771 100644 --- a/includes/OutputPage.php +++ b/includes/OutputPage.php @@ -2505,10 +2505,12 @@ class OutputPage { } continue; } - // Special handling for user and site groups; because users might change their stuff - // on-wiki like site or user pages, or user preferences; we need to find the highest + // Special handling for the user group; because users might change their stuff + // on-wiki like user pages, or user preferences; we need to find the highest // timestamp of these user-changable modules so we can ensure cache misses on change - if ( $group === 'user' || $group === 'site' ) { + // This should NOT be done for the site group (bug 27564) because anons get that too + // and we shouldn't be putting timestamps in Squid-cached HTML + if ( $group === 'user' ) { // Get the maximum timestamp $timestamp = 1; foreach ( $modules as $module ) { -- 2.20.1