Page MenuHomePhabricator

D12127.id29153.diff
No OneTemporary

D12127.id29153.diff

diff --git a/src/aphront/sink/AphrontPHPHTTPSink.php b/src/aphront/sink/AphrontPHPHTTPSink.php
--- a/src/aphront/sink/AphrontPHPHTTPSink.php
+++ b/src/aphront/sink/AphrontPHPHTTPSink.php
@@ -22,10 +22,11 @@
protected function emitData($data) {
echo $data;
- // Try to push the data to the browser. This has a lot of caveats around
- // browser buffering and display behavior, but approximately works most
- // of the time.
- flush();
+ // NOTE: We don't call flush() here because it breaks HTTPS under Apache.
+ // See T7620 for discussion. Even without an explicit flush, PHP appears to
+ // have reasonable behavior here: the echo will block if internal buffers
+ // are full, and data will be sent to the client once enough of it has
+ // been buffered.
}
protected function isWritable() {

File Metadata

Mime Type
text/plain
Expires
Sat, Nov 9, 7:47 PM (1 w, 3 d ago)
Storage Engine
blob
Storage Format
Encrypted (AES-256-CBC)
Storage Handle
6749434
Default Alt Text
D12127.id29153.diff (839 B)

Event Timeline