From changelog for 4.5:
"* added support for sending gzip compressed pages."
But I don`t get how it swich on. Does it works?
Searching drupal.org doesn`t help me.

So, I`ve made a small patch.

CommentFileSizeAuthor
gzip.patch.tar.bz26.7 KBNick@drupal.ru
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Bèr Kessels’s picture

Bèr Kessels CreditAttribution: Bèr Kessels commented

Small patch, looks nice.

but please, do not tar or zip your patches. That makes them hard to read, or try.

Bèr

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented

The comment in the release statement was referring to cached pages which are stored and sent as gzipped html.
I do not understand why we'd need your patch for sendign all content gzipped. Adding the relevant lines to the .htaccess file should suffice.

Nick@drupal.ru’s picture

Nick@drupal.ru CreditAttribution: Nick@drupal.ru commented

Oh... It`s doen`t work with cache enabled.

killes, hm...
when I just add
zlib.output_compression On
in .htaccess, compression works, but there is nothing about compression in headers (for authenticated users, not anonymous), so html-page has broken. Thats why I`ve tried to make this patch

Now I see, that compression works by default, but only for cache data (in 4.5).
And, probably, when
zlib.output_compression On
cache data compress twice.

I just want, all data to be compressed.

So, my patch works if only cache disabled.
Sorry.

killes@www.drop.org’s picture

killes@www.drop.org CreditAttribution: killes@www.drop.org commented

I think that zlib.output compression should check if the output is already compressed. Drupal does send the right headers, I think.
mod_gzip works fine with Drupal's compressed cache.