php / php/php-src

Gzip output handler - `ob_end_flush()` must not call `flush()`

Offen
#13,798 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bug Category: Output Extension: zlib Status: Needs Triage
Vorherrschende Sprache
C
Sterne
40.4k
Forks
8.1k
Ø Merge
2 T. 13 Std.
Gemergte PRs (30 T.)
96

Beschreibung

Description

This issue is problematic only when zlib.output_compression is enabled.

Currently, when zlib.output_compression is enabled like:

php -S 127.0.0.1:81 -d zlib.output_compression=1 -t .

(but also tested if enabled when used /w Apache /wo mod_deflate)

and ob_end_flush() is called, no headers cannot be send anymore even if the nothing was output.

Currently, we had to use the following code

while (ob_get_level() > 0) {
    if (ob_get_length() === 0) {
        ob_end_clean();
    } else {
        ob_end_flush();
    }
}

to workaround.

When zlib.output_compression is NOT enabled, the following code:

while (ob_get_level() > 0) {
    ob_end_flush();
}

is working (allows to send additional headers).

Because of this inconsistency, I belive php gzip output handler behaviour should be fixed, when output buffer is empty, to not flush implicitly.

PHP Version

any

Operating System

any

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Reproduziere das Verhalten mit dem bereitgestellten PHP CLI-Befehl, indem du einen leeren Ausgabepuffer mit einem Puffer vergleichst, der Ausgabe enthält, und prüfst, ob Header noch gesendet werden können. Untersuche anschließend den ob_end_flush()-Pfad und den zlib-Ausgabehandler; die Aufgabe ist abgeschlossen, wenn ein leerer Puffer nicht implizit flushen lässt, während eine nicht leere Ausgabe ihr erwartetes Verhalten beibehält.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
c, php
Bereich
backend
Issue-Typ
Bug
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.