-
Notifications
You must be signed in to change notification settings - Fork 108
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
JsSse in PHP-FPM setup requires special configuration #2178
Comments
This seems to be not-output buffer related. Can anything from atk4/ui be done? I tried to find some solutions and it seems sending at leasst 4096 bytes per each JSSE output can help, can you evaluate the possibilities? https://gist.github.com/hranicka/368dfbf3df971f021fa70047eb894058 https://syntaxfix.com/question/45690/how-to-flush-output-after-each-echo-call |
The PR in #2175 generally works. Only issue is that in FPM setup by default flush is not working (mod_fcgi) unless you activate it. Also you cannot reconfigure output buffer programmatically so you are stuck with an output buffer of 4096. If you reconfigure PHP-FPM to allow flush as in above configuration, then Atk4 Ui works out of the box |
great
we can account for that in SSE and always append 4kiB dummy string working FMP/FCGI repro:
and
|
I conducted a testing with various webservers:
Apache /w PHP FCGI is problematic as the response is bufferent by Apache and Based on https://stackoverflow.com/questions/30707792/how-to-disable-buffering-with-apache2-and-mod-proxy-fcgi#36298336 and testing, I found out that However, when Apache is configured to compress the output (using The question is, is it worth adding 4KiB of overheader to cover Apache /w PHP FCGI (and even only when configured to not compress all files)? Apache issue: https://bz.apache.org/bugzilla/show_bug.cgi?id=68827 |
Situation:
Solution:
Suggestions to limit flushing packets only on selected scripts or directories are welcome. This way you can keep output buffering for scripts that are non-Sse.
Also see:
php/php-src#12785 (comment)
The text was updated successfully, but these errors were encountered: