Ticket #4342 (closed defect (bug): fixed)

Opened 5 years ago

Last modified 4 years ago

gzhandler warning

Reported by: F3rn4nd0 Owned by: ryan
Priority: normal Milestone: 2.5
Component: Optimization Version: 2.2
Severity: normal Keywords: has-patch commit
Cc:

Description

Sometimes I can see a warning in the upper side like this: Warning: ob_start(): output handler 'ob_gzhandler' cannot be used after 'URL-Rewriter related to file functions.php

I found a patch using:

  if ( extension_loaded('zlib') )
      {
         ob_end_clean();
         ob_start('ob_gzhandler');
      }
   }

instead of

      if ( extension_loaded('zlib') )
      {
         ob_start('ob_gzhandler');
      }
   }

It seems to work

Attachments

4342.r6563.diff Download (505 bytes) - added by darkdragon 4 years ago.
Patch based off of r6563

Change History

  • Milestone changed from 2.4 to 2.2.1

Makes sense. Possibly another output buffer had started.

  • Milestone changed from 2.2.1 to 2.2.2
  • Milestone changed from 2.2.2 to 2.3

It is possible to check as to whether another output buffer had started?

Oh, ob_end_clean() probably shouldn't be used if another buffer had started. Unless you wish to discard the contents.

Patch based off of r6563

  • Keywords has-patch needs-testing added

Needs testing (unit testing) on whether or not an notice is thrown if there is no output buffer active when ob_clean_flush() is called.

E_NOTICE has to be on in order to see if there is a notice.

  • Keywords commit added; needs-testing removed

Can this go in?

comment:9   ryan4 years ago

Let's just get rid of gzip_compression(). It's caused too many problems for such a small bit of code.

  • Owner changed from anonymous to ryan

Yeah... plus, this is something that is much better if activated by a server admin at the PHP or HTTP server level. +1 for axing it. One less option!

  • Status changed from new to closed
  • Resolution set to fixed

(In [6775]) Remove gzip_compression(). Leave it to the server. fixes #4342

Note: See TracTickets for help on using tickets.