-
Notifications
You must be signed in to change notification settings - Fork 340
Open
Description
Please be advised that these lines on logging in the proposed configuration should be removed/changed:
logging
access_log /var/log/nginx/wordpress.access.log;
error_log /var/log/nginx/wordpress.error.log;
Let me explain the reasons behind.
a) it overrides the default logging format of dockerized nginx which means "$http_x_forwarded_for" is not logged
b) give the false impression that log files are saved to disk when in fact they are just sent to docker due to the way the nginx container is set up
Proposal: remove these lines or add at least the word main to the accesslog line:
access_log /var/log/nginx/wordpress.access.log main;
Metadata
Metadata
Assignees
Labels
No labels