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

Add an option to disable/redirect fpm access logs #374

Merged

Conversation

RadeJR
Copy link
Contributor

@RadeJR RadeJR commented May 31, 2024

Since the introduction of healthchecks in new images, my logs are very crowded with /healthcheck endpoint so I added an option to change access log file with environment variable. For my use case, I don't really need fpm access logs, for laravel they are just requests to /index.php route anyway, nginx access logs show more info, in that case we don't need duplicates.

@jaydrogers
Copy link
Member

Thanks for pointing this out! I ended up removing the access log all together unless it's in Debug Mode.

I silenced the Apache Access Log for the healthcheck as well 👍

@jaydrogers jaydrogers merged commit 11e8235 into serversideup:main Jun 19, 2024
1 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants