MDL-40002 Files: ETag value must be a quoted-string.
ETag value must be double quoted, http://tools.ietf.org/html/rfc2616#section-3.11. More details about potential impacts in MDL-39832: shortly, at least when using HTTP Partial GET Requests (HTTP Status: 206) plus X-Sendfile HTTP Header. In this commit, the impact is to be RFC2616 compliant when using the ETag Header.
This commit is contained in:
+3
-3
@@ -228,7 +228,7 @@ function css_send_ie_css($themename, $rev, $etag, $slasharguments) {
|
||||
$css .= "\n@import url($relroot/styles.php?theme=$themename&rev=$rev&type=theme);";
|
||||
}
|
||||
|
||||
header('Etag: '.$etag);
|
||||
header('Etag: "'.$etag.'"');
|
||||
header('Content-Disposition: inline; filename="styles.php"');
|
||||
header('Last-Modified: '. gmdate('D, d M Y H:i:s', time()) .' GMT');
|
||||
header('Expires: '. gmdate('D, d M Y H:i:s', time() + $lifetime) .' GMT');
|
||||
@@ -254,7 +254,7 @@ function css_send_ie_css($themename, $rev, $etag, $slasharguments) {
|
||||
function css_send_cached_css($csspath, $etag) {
|
||||
$lifetime = 60*60*24*60; // 60 days only - the revision may get incremented quite often
|
||||
|
||||
header('Etag: '.$etag);
|
||||
header('Etag: "'.$etag.'"');
|
||||
header('Content-Disposition: inline; filename="styles.php"');
|
||||
header('Last-Modified: '. gmdate('D, d M Y H:i:s', filemtime($csspath)) .' GMT');
|
||||
header('Expires: '. gmdate('D, d M Y H:i:s', time() + $lifetime) .' GMT');
|
||||
@@ -312,7 +312,7 @@ function css_send_unmodified($lastmodified, $etag) {
|
||||
header('Expires: '. gmdate('D, d M Y H:i:s', time() + $lifetime) .' GMT');
|
||||
header('Cache-Control: public, max-age='.$lifetime);
|
||||
header('Content-Type: text/css; charset=utf-8');
|
||||
header('Etag: '.$etag);
|
||||
header('Etag: "'.$etag.'"');
|
||||
if ($lastmodified) {
|
||||
header('Last-Modified: '. gmdate('D, d M Y H:i:s', $lastmodified) .' GMT');
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user