[obsolete] RawAction: Vary on the usual headers.patch
No One
Actions

Authored By

	Anomie
	Jan 31 2016, 9:11 PM

Size

1 KB

Referenced Files

None

Subscribers

None

[obsolete] RawAction: Vary on the usual headers.patch
View Options

	From a9738aed47eb80a6b85f4fa970a02f8f1dd1340c Mon Sep 17 00:00:00 2001
	From: Brad Jorsch <bjorsch@wikimedia.org>
	Date: Sun, 31 Jan 2016 15:43:00 -0500
	Subject: [PATCH] RawAction: Vary on the usual headers

	This avoids edge cases where the user isn't logged in but we still need
	varying for proper cache behavior.

	Bug: T125283
	Change-Id: I43cde3a48371e62a16bda1291b1b51986e60fe4c
	---
	includes/actions/RawAction.php \| 6 ++++++
	1 file changed, 6 insertions(+)

	diff --git a/includes/actions/RawAction.php b/includes/actions/RawAction.php
	index b371848..6cc5e0b 100644
	--- a/includes/actions/RawAction.php
	+++ b/includes/actions/RawAction.php
	@@ -80,6 +80,12 @@ class RawAction extends FormlessAction {
	}
	}

	+ // Set standard Vary headers so cache varies on cookies and such (T125283)
	+ $response->header( $this->getOutput()->getVaryHeader() );
	+ if ( $config->get( 'UseKeyHeader' ) ) {
	+ $response->header( $this->getOutput()->getKeyHeader() );
	+ }
	+
	$response->header( 'Content-type: ' . $contentType . '; charset=UTF-8' );
	// Output may contain user-specific data;
	// vary generated content for open sessions on private wikis
	--
	2.7.0

File Metadata

Mime Type: text/x-diff
Storage Engine: blob
Storage Format: Raw Data
Storage Handle: 3305418
Default Alt Text: [obsolete] RawAction: Vary on the usual headers.patch (1 KB)

Attached To	Mode
T125283: Users occasionally logged in as different users after SessionManager deployment	Attached	Detach File

Event Timeline

Anomie updated the name for this file from "RawAction: Vary on the usual headers.patch" to "[obsolete] RawAction: Vary on the usual headers.patch".Feb 23 2016, 3:50 PM

[obsolete] RawAction: Vary on the usual headers.patchNo OneActions

[obsolete] RawAction: Vary on the usual headers.patchView Options

File Metadata

Event Timeline

[obsolete] RawAction: Vary on the usual headers.patch
No One
Actions

[obsolete] RawAction: Vary on the usual headers.patch
View Options