Skip to content
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

Docker: Apache overrides were not enough #1873

Merged
merged 1 commit into from Apr 26, 2018

Conversation

Projects
None yet
1 participant
@Alkarex
Copy link
Member

commented Apr 26, 2018

It looks like overrides of CustomLog, ErrorLog, Listen did not do the
job fully.
Updated to comment out those lines in httpd.conf.
Fixes the fact that logs were kept in /var/log/apache2/ (instead of
being only given via STDOUT to Docker logs) and the "internal dummy
connection" bug over IPv6 (Docker is typically only IPv4).

Edit: this is because these Apache instructions are allowed to be repeated multiple times

Apache overrides were not enough
It looks like overrides of CustomLog, ErrorLog, Listen did not do the
job fully.
Updated to comment out those lines in httpd.conf.
Fixes the fact that logs were kept in /var/log/apache2/ (instead of
being only given via STDOUT to Docker logs) and the "internal dummy
connection" bug over IPv6 (Docker is typically only IPv4).

@Alkarex Alkarex added this to the 1.11.0 milestone Apr 26, 2018

@Alkarex Alkarex changed the title Apache overrides were not enough Docker: Apache overrides were not enough Apr 26, 2018

@Alkarex Alkarex merged commit 90998c2 into FreshRSS:dev Apr 26, 2018

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@Alkarex Alkarex deleted the Alkarex:docker-apache-fixes branch Apr 26, 2018

Alkarex added a commit that referenced this pull request Apr 26, 2018

@Alkarex Alkarex referenced this pull request May 28, 2018

Merged

FreshRSS 1.11.0 #1902

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.