system/files/foo prints gibberish (double-gzencoded page cache) for anonymous users.

CommentFileSizeAuthor
#8 1165958-double-gzencode.patch5.71 KBmfb
#7 1165958-double-gzencode.patch3.2 KBmfb
#4 1165958-double-gzencode.patch3.14 KBmfb
#3 double-gzencode.patch1.82 KBmfb
file_download.patch636 bytesmfb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Version: 7.x-dev » 8.x-dev

patch also applies on 8.x branch

Status: Needs review » Needs work

The last submitted patch, file_download.patch, failed testing.

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Title: system/files/foo gibberish if cache and page_compression are enabled » gibberish on certain paths if cache and page_compression are enabled
Component: file system » base system
Status: Needs work » Needs review
FileSize
double-gzencode.patch1.82 KB

I found a few other places where this issue crops up, including image and comment modules. Let's see if the fix for image module also fixes the test failure.

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
FileSize
1165958-double-gzencode.patch3.14 KB

Added a test for the system/files case of this bug that fails without the patch.

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Issue tags: +Needs backport to D7
mfb’s picture

mfb
they or he
CreditAttribution: mfb commented

Also found that profile module suffers from this bug. 

  drupal_not_found();
  return;

and 

  drupal_access_denied();
  return;

instead of 

  return MENU_NOT_FOUND;

and 

  return MENU_ACCESS_DENIED;
mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
FileSize
1165958-double-gzencode.patch3.2 KB

rerolled.

mfb’s picture

mfb
they or he
CreditAttribution: mfb commented
Version: 8.x-dev » 7.x-dev
FileSize
1165958-double-gzencode.patch5.71 KB

I believe these issues have all been separately fixed in d8 so I am moving this back to d7.