MCS should quote the ETag
Closed, ResolvedPublic
Actions

Assigned To

Authored By

	• Pchelolo
	Nov 16 2016, 7:18 PM

Description

The last unresolved piece in T148590 is that MCS currently doesn't double-quote the ETag header, which is should according to http spec.

So etag should be transformed from 123/uuid to "123/uuid".

	Subject	Repo	Branch	Lines +/-
	Quote the ETag being set and fix it up in some endpoints	mediawiki/services/mobileapps	master	+38 -77

Mentioned In: T148590: MCS doesn't set ETag if 'accept-encoding' header is provided.
Mentioned Here: rGMODae1d3c2321a9: Update mobileapps to f5d9d86
T148590: MCS doesn't set ETag if 'accept-encoding' header is provided.

Change 322084 had a related patch set uploaded (by Mobrovac):
Quote the ETag being set

Change 322084 merged by jenkins-bot:
Quote the ETag being set and fix it up in some endpoints

deploy/2016-12-14/ae1d3c23