Page Menu
Home
Phabricator
Search
Configure Global Search
Log In
Files
F14029128
D12127.id29153.diff
No One
Temporary
Actions
View File
Edit File
Delete File
View Transforms
Subscribe
Mute Notifications
Award Token
Flag For Later
Size
839 B
Referenced Files
None
Subscribers
None
D12127.id29153.diff
View Options
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
Details
Attached
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)
Attached To
Mode
D12127: Don't call flush() when emitting responses
Attached
Detach File
Event Timeline
Log In to Comment