Skip to content

Commit

Permalink
Ensure fast flush of HTTP headers and HTML head
Browse files Browse the repository at this point in the history
The fast flush optimisation done in
FreshRSS#1133 does not seem to work
anymore (need to check if it is related to a PHP version).
Work-around when PHP flush() is not working
Can be tested by adding a `sleep(5);` after:

https://github.com/FreshRSS/FreshRSS/blob/ee902ee7c4370421802768c3105ba269a4f97b16/app/layout/layout.phtml#L27
Follow-up of the performance checks of
FreshRSS#2040
  • Loading branch information
Alkarex committed Oct 13, 2018
1 parent ee902ee commit bec6ad2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 0 deletions.
1 change: 1 addition & 0 deletions Docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ WORKDIR ${FRESHRSS_ROOT}

COPY . ${FRESHRSS_ROOT}
COPY ./Docker/*.Apache.conf /etc/apache2/conf.d/
COPY ./Docker/*.php.ini /etc/php7/conf.d/

RUN sed -r -i "/^[ ]*(CustomLog|ErrorLog|Listen) /s/^/#/" /etc/apache2/httpd.conf && \
echo "17,37 * * * * php ${FRESHRSS_ROOT}/app/actualize_script.php 2>&1 | tee /tmp/FreshRSS.log" >> \
Expand Down
2 changes: 2 additions & 0 deletions Docker/FreshRSS.php.ini
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
; Keep below ~2600 to ensure fast headers when flush() is not working
output_buffering = 2500

0 comments on commit bec6ad2

Please sign in to comment.